Skip to content

merge main which includes process changes into v1.1 #784

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

Merged
merged 2 commits into from
Jul 31, 2021

Conversation

kdenhartog
Copy link
Member

@kdenhartog kdenhartog commented Jul 29, 2021

This PR is here to bring main and v1.1 back in alignment. The specific commit I'm looking to bring into V1.1 is the changes to the readme which are defining the process of how we will go about handling changes in the maintenance WG as agreed to via the CCG and merged already in main.


Preview | Diff

* Add PR review process for 2021.

* Avoid GitHub id auto-linking.

Co-authored-by: David I. Lehn <[email protected]>

* Update README.md

Co-authored-by: Manu Sporny <[email protected]>

Co-authored-by: Manu Sporny <[email protected]>
Co-authored-by: David I. Lehn <[email protected]>
Co-authored-by: Brent Zundel <[email protected]>
@kdenhartog kdenhartog added clarification Non-normative clarifications of spec text v1.1 labels Jul 29, 2021
@kdenhartog kdenhartog force-pushed the kdh/v1.1-add-process branch from bca9e00 to 16f2e34 Compare July 29, 2021 04:48
@kdenhartog kdenhartog changed the title Rebase main with process changes into v1.1 merge main which includes process changes into v1.1 Jul 29, 2021
@kdenhartog
Copy link
Member Author

@TallTed this should fix that issue on #783 to separate the two out

- 1 substantive update (v1.2?)
- VC Test Suite Refactoring
- Start planning VC v2 Work, request a rechartering 3-6 months before end of
year to keep VC WG functioning.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
year to keep VC WG functioning.
year to keep VC WG functioning

- [v1] Fixing a specific bug
- [v1] Update examples in the spec to make them modern
- [v2] VC `@context` needs updating, possibly with security vocab modularized
into smaller components instead of all included into a large context file.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
into smaller components instead of all included into a large context file.
into smaller components instead of all included into a large context file

@kdenhartog
Copy link
Member Author

@TallTed I agree with your suggested changes, but I'm going to leave them out in this scenario in order to keep alignment between main and V1.1 as close as possible. Main reason being I just want this in so that when commits are targeted at main and I need to update to V1.1 I don't want any wonky git overrides to slow things down. We can merge these separately onto main first and then merge them downstream to this one after if you'd like.

@kdenhartog kdenhartog merged commit b1ea54d into v1.1 Jul 31, 2021
@kdenhartog kdenhartog deleted the kdh/v1.1-add-process branch July 31, 2021 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clarification Non-normative clarifications of spec text
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants