Skip to content
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

Migrate Data Integrity section to dedicated specification. #900

Merged
merged 1 commit into from
Aug 14, 2022

Conversation

msporny
Copy link
Member

@msporny msporny commented Jul 30, 2022

This PR migrates the data integrity section into a dedicated specification.

DO NOT MERGE this until the Data Integrity specification is migrated into the VCWG.


Preview | Diff

@msporny msporny added the substantive Substantive changes to the specification. label Jul 30, 2022
@msporny msporny requested review from OR13 and iherman July 30, 2022 15:13
@iherman
Copy link
Member

iherman commented Jul 30, 2022

Is your intention of putting this into the spec before it goes out to FPWD? I would prefer not; do that only when a FPWD is out for DI (using then a proper reference).

@msporny
Copy link
Member Author

msporny commented Jul 30, 2022

Is your intention of putting this into the spec before it goes out to FPWD? I would prefer not; do that only when a FPWD is out for DI (using then a proper reference).

Nope, all PRs I've raised since last Wednesday are intended to be for AFTER the FPWD.

That is, this FPWD is final and will not change:

https://w3c.github.io/vc-data-model/WD/2022-08-11/

Copy link
Contributor

@peacekeeper peacekeeper left a comment

Choose a reason for hiding this comment

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

+1 to this after FPWD is out and Data Integrity has been migrated into VCWG.

@msporny msporny added DO NOT MERGE PR contains something that should not be merged. and removed do-not-merge labels Aug 3, 2022
@msporny msporny removed the DO NOT MERGE PR contains something that should not be merged. label Aug 14, 2022
@msporny
Copy link
Member Author

msporny commented Aug 14, 2022

Substantive, multiple reviews, VCDM v2.0 FPWD is published, no objections, merging.

@msporny msporny merged commit 2159bb3 into main Aug 14, 2022
@msporny msporny deleted the msporny-move-di branch August 14, 2022 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
substantive Substantive changes to the specification.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants