Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@marcelamelara
Copy link
Contributor

This PR adds a short description clarifying when the attestation spec version is rev'd and how that relates to tagged releases.

@marcelamelara marcelamelara requested a review from a team as a code owner May 7, 2024 17:22
Copy link
Contributor

@TomHennen TomHennen left a comment

Choose a reason for hiding this comment

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

Thanks, this is great!

### Examples

- Attestation Framework tagged release v1.0.2 (PATCH version) incorporates
refinements to the predicate specification process, a new predicate type,
Copy link
Member

Choose a reason for hiding this comment

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

IMO it is a bit confusing for semantic versioning of the spec to not apply to predicate changes (which I agree with) while also saying a spec version ships with a new predicate. Maybe the version of the spec should entirely omit predicates and it's just a matter of convenience that predicates live alongside the spec?

This does get a bit confusing with versioning of the code gen libraries though, as you highlight here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I see what you mean. The intent here isn't to say that none of the things under the PATCH version category really affect the core spec. Maybe we can take out the mention of predicate types altogether and say that PATCH versions are only for doc edits and language binding changes?

Copy link
Member

Choose a reason for hiding this comment

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

I think that might be better!

Copy link
Member

Choose a reason for hiding this comment

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

I also realized that language bindings are going to be fine; breaking changes in a predicate would be effectively a separate model with its own proto from the previous one.

Copy link
Member

@pxp928 pxp928 left a comment

Choose a reason for hiding this comment

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

LGTM!

@pxp928 pxp928 merged commit 36c1129 into in-toto:main May 28, 2024
@marcelamelara marcelamelara deleted the update-versioning branch August 30, 2024 19:17
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.

4 participants