Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update contracts #152

Closed
wants to merge 4 commits into from
Closed

chore: update contracts #152

wants to merge 4 commits into from

Conversation

leovct
Copy link
Member

@leovct leovct commented Nov 7, 2023

Description

  • Generate a new version of the ERC20 bytecode. It was always reverting during the deployment.
  • The contracts/ folder is now a foundry project which makes it easier to handle dependencies.
  • ERC20.sol and ERC721.sol have been rewritten to use openzeppelin standards.
  • Add new CI job to make sure contracts can be built using forge.

Jira / Linear Tickets

Testing

  • Deploy ERC20 smart contract from bytecode.
$ cast send \
    --from 0x85dA99c8a7C2C95964c8EfD687E95E632Fc533D6 \
    --private-key 0x42b6e34dc21598a807dc19d7784c71b2a7a01f6480dc6f58258f78e539f1a1fa \
    --rpc-url localhost:8545 \
    --json \
    --create \
    "$(cat ./contracts/src/tokens/ERC20.bin)" | jq
✅
  • go run main.go loadtest --mode erc20 --verbosity 600 --requests 100 --rate-limit 100
  • go run main.go loadtest --mode erc721 --verbosity 600 --requests 100 --rate-limit 100
  • go run main.go loadtest uniswapv3 --verbosity 600 --requests 100 --rate-limit 100
  • go run main.go loadtest --mode random --verbosity 600 --requests 100 --rate-limit 100

@leovct leovct force-pushed the chore/update-contracts-2 branch from 1b826d8 to f3e4b0b Compare November 7, 2023 15:13
@leovct leovct requested a review from a team November 7, 2023 18:11
@leovct
Copy link
Member Author

leovct commented Nov 20, 2023

PR closed because of conflicts. See #162.

@leovct leovct closed this Nov 20, 2023
@leovct leovct deleted the chore/update-contracts-2 branch November 20, 2023 16:09
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.

1 participant