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

Revert "stop testing broken upstream version-2-0" #6506

Merged
merged 2 commits into from
Aug 23, 2024
Merged

Conversation

arnetheduck
Copy link
Member

Reverts #6499

@arnetheduck
Copy link
Member Author

nim-lang/Nim#24002

Copy link

Unit Test Results

         9 files  ±0    1 334 suites  ±0   34m 13s ⏱️ + 2m 28s
  5 064 tests ±0    4 716 ✔️ ±0  348 💤 ±0  0 ±0 
20 991 runs  ±0  20 587 ✔️ ±0  404 💤 ±0  0 ±0 

Results for commit 412dd6a. ± Comparison against base commit 77c36b3.

@arnetheduck arnetheduck marked this pull request as draft August 22, 2024 12:14
@tersec
Copy link
Contributor

tersec commented Aug 22, 2024

No objections to merging this once the issue is resolved

@arnetheduck
Copy link
Member Author

so per the comments in the PR, this is actually a latent bug in our code exposed by additional strictness - ie the template never worked as intended basically, and prints None in the logs.

@tersec
Copy link
Contributor

tersec commented Aug 23, 2024

In which case, fixing that code in nimbus-eth2 works also

@arnetheduck arnetheduck marked this pull request as ready for review August 23, 2024 09:34
@arnetheduck
Copy link
Member Author

seems to work

@tersec tersec enabled auto-merge (squash) August 23, 2024 10:07
@tersec tersec merged commit 42c4ea1 into unstable Aug 23, 2024
12 checks passed
@tersec tersec deleted the revert-6499-y9h branch August 23, 2024 10:18
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