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

RFC37: ckb2021 overview #242

Merged
merged 6 commits into from
Mar 18, 2022
Merged

RFC37: ckb2021 overview #242

merged 6 commits into from
Mar 18, 2022

Conversation

doitian
Copy link
Member

@doitian doitian commented Jul 14, 2021

No description provided.

@doitian doitian requested a review from a team as a code owner July 14, 2021 09:19
@doitian doitian requested review from quake and knwang July 14, 2021 09:19
@doitian doitian marked this pull request as draft July 14, 2021 09:19
@doitian doitian added the ckb2021 Hard fork scheduled in 2021 label Jul 15, 2021
@doitian doitian mentioned this pull request Jul 23, 2021
Open
14 tasks
@doitian doitian changed the title ckb2021 overview RFC37: ckb2021 overview Jul 23, 2021
@doitian doitian marked this pull request as ready for review July 24, 2021 01:20
@doitian doitian mentioned this pull request Sep 3, 2021
Closed
janx
janx reviewed Mar 1, 2022
View reviewed changes
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
@doitian @janx
Apply suggestions from code review
6403a6f 
Co-authored-by: busyforking <5958+janx@users.noreply.github.com>
@doitian doitian requested a review from janx March 2, 2022 03:47
janx
janx reviewed Mar 2, 2022
View reviewed changes
rfcs/0037-ckb2021/0037-ckb2021.md Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
rfcs/0037-ckb2021/0037-ckb2021.md Outdated Show resolved Hide resolved
@doitian @janx
Apply suggestions from code review
dbc28ee 
Co-authored-by: busyforking <5958+janx@users.noreply.github.com>
@doitian doitian requested review from janx and removed request for quake March 3, 2022 06:38
@doitian doitian requested review from zhangsoledad and removed request for knwang March 4, 2022 02:16
@doitian
Copy link
Member Author

doitian commented Mar 4, 2022

Assigned reviewers: @janx @zhangsoledad

@doitian doitian added the s:fcp Final Comment Period. PR is merged when there's no comments in 14 days. label Mar 14, 2022
@doitian doitian merged commit f8fc1f5 into nervosnetwork:master Mar 18, 2022
@doitian doitian deleted the ckb2021-overview branch March 18, 2022 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Keith-CY Keith-CY Keith-CY left review comments

@janx janx janx approved these changes

@zhangsoledad zhangsoledad zhangsoledad approved these changes

Assignees
No one assigned
Labels
ckb2021 Hard fork scheduled in 2021 s:fcp Final Comment Period. PR is merged when there's no comments in 14 days.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@doitian @janx @zhangsoledad @Keith-CY