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.

Sign up for GitHub

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

Jump to bottom

Describe and hoist stripped state to a first-class citizen #3606

Merged
merged 4 commits into from Jan 5, 2022
Merged

Describe and hoist stripped state to a first-class citizen #3606

merged 4 commits into from Jan 5, 2022

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Dec 29, 2021 •
edited by github-actions bot

This was referenced Dec 29, 2021
Merged
Closed
@turt2live turt2live requested a review from a team December 29, 2021 23:03
@turt2live turt2live added this to Needs idea feedback / initial review in Spec Core Team Backlog via automation Dec 29, 2021
@turt2live turt2live moved this from Needs idea feedback / initial review to Requires spec review (post-FCP) in Spec Core Team Backlog Dec 29, 2021
@turt2live turt2live mentioned this pull request Dec 31, 2021
Merged
uhoreg
uhoreg approved these changes Jan 5, 2022
View reviewed changes
Copy link
Member

@uhoreg uhoreg left a comment

Choose a reason for hiding this comment

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

Assuming @clokep doesn't have further suggestions, lgtm

@turt2live turt2live requested a review from clokep January 5, 2022 03:23
clokep
clokep approved these changes Jan 5, 2022
View reviewed changes
Copy link
Contributor

@clokep clokep left a comment

Choose a reason for hiding this comment

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

No more suggestions from me, very happy to see this formally declared in a single spot!

@turt2live turt2live merged commit 56532dd into main Jan 5, 2022
@turt2live turt2live deleted the travis/spec/stripped-state-3173 branch January 5, 2022 17:58
@turt2live turt2live moved this from Requires spec review (post-FCP) to Done to some definition in Spec Core Team Backlog Jan 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@clokep clokep clokep approved these changes

@uhoreg uhoreg uhoreg approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Spec Core Team Backlog
  
Done to some definition
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

How to incorporate MSC3173 (stripped state for all) into the spec?
3 participants
@turt2live @clokep @uhoreg