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

CI fix: verify-near-headers, verify-near-proofs #898

Merged
merged 18 commits into from
Oct 23, 2023
Merged

CI fix: verify-near-headers, verify-near-proofs #898

merged 18 commits into from
Oct 23, 2023

Conversation

UrAvgDeveloper
Copy link
Contributor

@UrAvgDeveloper UrAvgDeveloper commented May 9, 2023

  • Fixed data issue for failing CIs - verify-near-headers and verify-near-proofs
  • Updated nearbridge tests which were skipped along with header data files used

To fix E2E tests following are the changes implemented:

  • Changed ganache version and fixed ganache start script to use automine in some time
  • Changed near-core version to 1.31.0, because of use of IPv6 port in 1.32.0. Links to verify are this and this. Main issue is GitHub actions not supporting IPv6 ports, the issue for reference.

@UrAvgDeveloper UrAvgDeveloper marked this pull request as ready for review May 12, 2023 11:19
@UrAvgDeveloper UrAvgDeveloper requested a review from karim-en as a code owner May 12, 2023 11:19
@UrAvgDeveloper UrAvgDeveloper requested a review from sept-en May 12, 2023 11:22
@UrAvgDeveloper UrAvgDeveloper marked this pull request as draft May 15, 2023 12:41
@UrAvgDeveloper UrAvgDeveloper force-pushed the ci-fix branch 2 times, most recently from d42b211 to eeb19aa Compare June 5, 2023 08:01
@UrAvgDeveloper UrAvgDeveloper marked this pull request as ready for review June 12, 2023 06:46
@UrAvgDeveloper UrAvgDeveloper requested a review from olga24912 June 12, 2023 06:46
@karim-en karim-en merged commit 3420dc7 into master Oct 23, 2023
22 of 24 checks passed
@karim-en karim-en deleted the ci-fix branch October 23, 2023 19:11
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