-
Notifications
You must be signed in to change notification settings - Fork 1.4k
feat: update version and apis #6587
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
base: master
Are you sure you want to change the base?
Conversation
symphony/chain.json
Outdated
"recommended_version": "v1.0.0", | ||
"compatible_versions": ["v1.0.0"], | ||
"recommended_version": "v1.0.7", | ||
"compatible_versions": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Version updates must also be reflected in the verions.json file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i've updated versions.json to include 1.0.7. i've also removed from compatible versions so only the included tags for full releases (the ones listed) are included
}, | ||
"consensus": { | ||
"type": "cometbft", | ||
"version": "v0.38.12" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Path: consensus.version, Reason: Value mismatch, In "codebase": 0.38.11, In currentVersion: v0.38.12
{ | ||
"name": "v1.0.7", | ||
"tag": "v1.0.7", | ||
"height": 3333145, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This don't look to be the right block height. It's already used for previous versions.
(we'll be adding validation that the same block height isn't used on multiple version objects)
No description provided.