Skip to content

fix: change symbol from goqxmp to qxmp#8816

Merged
damodarnaik699 merged 1 commit into
masterfrom
SCAAS-9437
May 20, 2026
Merged

fix: change symbol from goqxmp to qxmp#8816
damodarnaik699 merged 1 commit into
masterfrom
SCAAS-9437

Conversation

@damodarnaik699
Copy link
Copy Markdown
Contributor

@damodarnaik699 damodarnaik699 commented May 20, 2026

@damodarnaik699 damodarnaik699 requested a review from a team as a code owner May 20, 2026 09:37
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 20, 2026

SCAAS-9437

@damodarnaik699
Copy link
Copy Markdown
Contributor Author

@claude review this pr

@damodarnaik699
Copy link
Copy Markdown
Contributor Author

@copilot review this pr throughly

Copy link
Copy Markdown

@vipulchaursiya908 vipulchaursiya908 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vipulchaursiya908
Copy link
Copy Markdown

@claude can you give approval rating

Copy link
Copy Markdown
Contributor

Copilot AI commented May 20, 2026

@copilot review this pr throughly

Completed a thorough review on commit e312bc7. I verified symbol/underlying-asset/address updates are consistent across base.ts, erc20Coins.ts, ofcErc20Coins.ts, and naming-convention tests; no goqxmp references remain. Validation run: modules/statics targeted unit test (tokenNamingConvention.ts) passed, and modules/statics build passed. No additional code changes are required.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • download.cypress.io
    • Triggering command: /opt/hostedtoolcache/node/24.14.1/x64/bin/node /opt/hostedtoolcache/node/24.14.1/x64/bin/node index.js --exec install (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

@damodarnaik699 damodarnaik699 merged commit 6add7f1 into master May 20, 2026
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants