Skip to content

Conversation

@shariqnaiyer
Copy link
Collaborator

πŸ—’οΈ Description

Following the discussion in #309, I believe versioning the API endpoints from the get go is beneficial long term.

πŸ”— Related Issues or PRs

#309
#309 (comment)

βœ… Checklist

  • Ran tox checks to avoid unnecessary CI fails:
    uvx tox
  • Considered adding appropriate tests for the changes.

Copy link
Contributor

@ch4r10t33r ch4r10t33r left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@shariqnaiyer
Copy link
Collaborator Author

I have versioned /health as well to be consistent with Beacon-API

Copy link
Contributor

@KolbyML KolbyML left a comment

Choose a reason for hiding this comment

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

:shipit: looks good

@KolbyML KolbyML merged commit 690298d into leanEthereum:main Jan 24, 2026
10 checks passed
@unnawut unnawut added the specs Scope: Changes to the specifications label Jan 24, 2026
@unnawut unnawut added this to the pq-devnet-2 milestone Jan 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

specs Scope: Changes to the specifications

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants