Skip to content

Comments

Move jsons to spec-tests repo#602

Open
GalRogozinski wants to merge 4 commits intomainfrom
move-jsons
Open

Move jsons to spec-tests repo#602
GalRogozinski wants to merge 4 commits intomainfrom
move-jsons

Conversation

@GalRogozinski
Copy link
Contributor

Summary

  • Move generated spec-test JSONs to an external spec-tests sibling repo layout.
  • Update generators, test loaders, and state-comparison helpers to read/write the new path.
  • Extend CI to generate JSONs and sync them to spec-tests via PRs and main-branch merges.

Changes

  • Generators now write to ../spec-tests/{qbft,ssv,types} and clean/create those directories.
  • Test runners and state-comparison helpers resolve JSONs from the external layout.
  • CI:
    • checks out the spec-tests repo
    • on PRs: generates JSONs and opens/updates a matching PR in spec-tests
    • on main: syncs generated JSONs to spec-tests and merges the matching PR if present

CI Requirements

  • Repo variable: SPEC_TESTS_REPO (e.g. ssvlabs/spec-tests)
  • Secret: SPEC_TESTS_TOKEN (write access to spec-tests)

Notes

  • Generated JSONs are now out-of-repo by default; make generate-jsons still works.

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Feb 3, 2026

Too many files changed for review. (2296 files found, 150 file limit)

@GalRogozinski GalRogozinski changed the title Move Jsons to spec-tests repo Move jsons to spec-tests repo Feb 3, 2026
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