-
Notifications
You must be signed in to change notification settings - Fork 19
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
Appendix on Proof Sets/Chains: Clarity improvements #335
base: main
Are you sure you want to change the base?
Conversation
…eak into two sentences.
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.
Tweaks for clarity
…g improvements. Co-authored-by: Ted Thibodeau Jr <[email protected]>
…ding improvements. Co-authored-by: Ted Thibodeau Jr <[email protected]>
The issue was discussed in a meeting on 2025-02-26 List of resolutions:
View the transcript1.1. Appendix on Proof Sets/Chains: Clarity improvements (pr vc-data-integrity#335)See github pull request vc-data-integrity#335. Brent Zundel: Ok, that means DI has no issues not marked future / horizontal review. Manu Sporny: I don't know how I missed this one. Brent Zundel: It is a very small PR. Manu Sporny: I can merge it. Brent Zundel: We can mark this future and make it the first editorial one. Manu Sporny: Yes, I just marked it that way. Brent Zundel: Ok, good.
Brent Zundel: Finally, there is a PR to add the revision history and changes since we went into Candidate Rec. Manu Sporny: I suggest these are fairly editorial and if we make a proposal to move VC-JOSE-COSE to PR that we include text that says we'll merge the remaining PRs before the request is issued. Brent Zundel: I'm ok with that. I want to note that the implementation report ... the VC-JOSE-COSE test suite implementation report shows sufficient implementations of every feature. Manu Sporny: That's right. It hasn't been integrated into the test suite report but we have at least two implementers saying they implemented the "status message" mechanism and we can get that into the report before it goes out to the membership. Brent Zundel: Cool. With that, BitstringStatusList is also ready to be moved into PR. Manu Sporny: We should be explicit. Brent Zundel: I will add a parenthetical comment.
Dave Longley: +1.
Brent Zundel: I have said it before and will say it again right now, that set of documents reflects an absolutely enormous amount of work that this group has done. |
Editorial only. Informative Appendix. Fix wording to add missing explanatory text. It looks like some text got chopped on in the explanation of proof sets. This PR provides a fix.
Preview | Diff