Skip to content

Use specific hardhat version on openzeppelin and gp2 external tests #14938

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

Merged
merged 1 commit into from
Mar 14, 2024

Conversation

r0qs
Copy link
Member

@r0qs r0qs commented Mar 13, 2024

So we can enable those external tests in the CI again

@r0qs r0qs force-pushed the cancun-fix-zeppelin-gp2-tests branch from acf3512 to 8dc0094 Compare March 13, 2024 23:43
cameel
cameel previously approved these changes Mar 13, 2024
Copy link
Member

@cameel cameel left a comment

Choose a reason for hiding this comment

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

Needs a small tweak but otherwise looks fine as long as it succeeds in CI.

@r0qs r0qs force-pushed the cancun-fix-zeppelin-gp2-tests branch from 8dc0094 to c5b438e Compare March 13, 2024 23:48
@r0qs r0qs enabled auto-merge March 13, 2024 23:53
@cameel
Copy link
Member

cameel commented Mar 13, 2024

I'd be careful with auto-merge here. It will merge the PR even if non-required jobs fail. And the two you're changing are not required :)

This should be merged only if it passes all the tests.

@cameel cameel disabled auto-merge March 13, 2024 23:57
@nikola-matic
Copy link
Collaborator

Build is green across the board, so I'll be merging.

@nikola-matic nikola-matic merged commit b0ffccc into develop Mar 14, 2024
@nikola-matic nikola-matic deleted the cancun-fix-zeppelin-gp2-tests branch March 14, 2024 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants