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

Added ERC20 tests in CI #54

Merged
merged 8 commits into from
Dec 23, 2024
Merged

Added ERC20 tests in CI #54

merged 8 commits into from
Dec 23, 2024

Conversation

traiansf
Copy link
Member

@traiansf traiansf commented Dec 20, 2024

Finally, after a lot of trial and error, and with help and suggestions from Jyoti, Stephen and Virgil, this is ready for review.

If you have any suggestions, esp on improving times, they are more than welcome.

@traiansf traiansf force-pushed the erc20-tests-in-ci branch 3 times, most recently from a2ba817 to 876de85 Compare December 20, 2024 19:13
@traiansf traiansf self-assigned this Dec 20, 2024
@traiansf traiansf force-pushed the erc20-tests-in-ci branch 2 times, most recently from 4f6ba87 to 3e0d4c7 Compare December 20, 2024 19:43
@traiansf traiansf force-pushed the erc20-tests-in-ci branch 2 times, most recently from 47796bd to 7f2ff29 Compare December 21, 2024 18:44
Copy link

@sskeirik sskeirik left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

@sskeirik sskeirik merged commit b8b2856 into master Dec 23, 2024
3 checks passed
@sskeirik sskeirik deleted the erc20-tests-in-ci branch December 23, 2024 20:16
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.

2 participants