Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Sep 17, 2025

This PR resolves a critical inconsistency in token supply values that would cause deployment verification failures.

Problem

The deployment scripts and test files had different totalSupply values:

  • scripts/deploy.js and scripts/deploy-DE.js: 500,000 tokens
  • test/SEQICO.test.js: 1,000,000 tokens

This inconsistency would cause deployment verification to fail when the actual deployed contract supply doesn't match test expectations.

Solution

Updated all files to use 750,000 tokens as requested by @sonnyquinn24:

// Before
const totalSupply = ethers.parseEther("500000"); // deploy scripts
const totalSupply = ethers.parseEther("1000000"); // test file

// After  
const totalSupply = ethers.parseEther("750000"); // all files

Changes Made

  • scripts/deploy.js: Updated totalSupply from 500,000 to 750,000 tokens
  • scripts/deploy-DE.js: Updated totalSupply from 500,000 to 750,000 tokens
  • test/SEQICO.test.js: Updated totalSupply from 1,000,000 to 750,000 tokens
  • README.md: Updated documentation to reflect new 750,000 token supply

All files now consistently reference the same token supply value, ensuring deployment verification will pass and the contract behaves as expected in both deployment and testing scenarios.

Fixes #39.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copy link

github-actions bot commented Sep 17, 2025

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

  • package-lock.json

@Copilot Copilot AI changed the title [WIP] The totalSupply is set to 500 million tokens, but the test file expects 100 million tokens. This inconsistency will cause deployment verification to fail when compared against the test expectations. Fix token supply inconsistency: update to 750,000 tokens across all files Sep 17, 2025
@Copilot Copilot AI requested a review from sonnyquinn24 September 17, 2025 20:51
Copilot finished work on behalf of sonnyquinn24 September 17, 2025 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants