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

re-tweaked cryptosuite deliverables #40

Merged
merged 1 commit into from
Jan 19, 2022
Merged

re-tweaked cryptosuite deliverables #40

merged 1 commit into from
Jan 19, 2022

Conversation

TallTed
Copy link
Member

@TallTed TallTed commented Jan 14, 2022

same changes were submitted (and, I thought, applied) previously, but they weren't in main when I just checked, so here they are again


Preview | Diff

same changes were submitted (and, I thought, applied) previously, but they weren't in main when I just checked, so here they are again
@msporny
Copy link
Member

msporny commented Jan 14, 2022

same changes were submitted (and, I thought, applied) previously, but they weren't in main when I just checked, so here they are again

Apologies, don't know what happened there, but agree that these changes need to go in and were already approved.

@iherman iherman merged commit 6ebe911 into w3c:main Jan 19, 2022
@iherman
Copy link
Member

iherman commented Jan 19, 2022

The issue was discussed in a meeting on 2022-01-19

  • no resolutions were taken
View the transcript

5.3. re-tweaked cryptosuite deliverables (pr vc-wg-charter#40)

See github pull request vc-wg-charter#40.

Brent Zundel: final business for today, PR 40.
… significantly improves readability, any feedback on 40?.
… already approvals, if no objections, let's merge 40 today.

Ivan Herman: i would merge 40, and leave 41 for now?.

Brent Zundel: sounds good.
… raised an issue to track the concerns, looking forward to the conversation.
… any conversation we can have before we meet next, will help that conversation to be productive. i invite you all to engage this week as your schedules permit.
… thanks everyone! we have done a significant amount of work.

@TallTed TallTed deleted the patch-1 branch January 19, 2022 17:11
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.

None yet

5 participants