Skip to content
This repository has been archived by the owner on Aug 13, 2024. It is now read-only.

[KIP-149] Unified system contract management process #150

Merged
merged 26 commits into from
Dec 14, 2023
Merged

[KIP-149] Unified system contract management process #150

merged 26 commits into from
Dec 14, 2023

Conversation

hyeonLewis
Copy link
Contributor

KIP-149 is a proposal for a unified system contract management process
See this PR in markdown
Closes #149

@github-actions
Copy link

github-actions bot commented Sep 20, 2023 •
edited
Loading

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@hyeonLewis hyeonLewis marked this pull request as draft September 21, 2023 06:24
scott-klaytn
scott-klaytn suggested changes Sep 21, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
KIPs/kip-149.md Outdated Show resolved Hide resolved
KIPs/kip-149.md Outdated Show resolved Hide resolved
KIPs/kip-149.md Outdated Show resolved Hide resolved
hyunsooda
hyunsooda reviewed Sep 22, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
KIPs/kip-149.md Outdated Show resolved Hide resolved
KIPs/kip-149.md Outdated Show resolved Hide resolved
KIPs/kip-149.md Outdated Show resolved Hide resolved
KIPs/kip-149.md Outdated Show resolved Hide resolved
@hyeonLewis hyeonLewis marked this pull request as ready for review October 4, 2023 04:26
blukat29
blukat29 reviewed Oct 5, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
ian0371
ian0371 reviewed Oct 6, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
ian0371
ian0371 reviewed Oct 6, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
kjhman21
kjhman21 reviewed Dec 7, 2023
View reviewed changes
KIPs/kip-149.md Outdated Show resolved Hide resolved
@hyeonLewis hyeonLewis mentioned this pull request Dec 7, 2023
Merged
9 tasks
kjhman21
kjhman21 approved these changes Dec 11, 2023
View reviewed changes
Copy link
Collaborator

@kjhman21 kjhman21 left a comment

Choose a reason for hiding this comment

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

Thanks for the hard work @hyeonLewis !

@hyeonLewis
Copy link
Contributor Author

Thanks for your careful review @kjhman21. Please take a final look @ian0371 @blukat29 @aidan-kwon.

aidan-kwon
aidan-kwon approved these changes Dec 14, 2023
View reviewed changes
Copy link
Member

@aidan-kwon aidan-kwon left a comment

Choose a reason for hiding this comment

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

Thanks!

ian0371
ian0371 approved these changes Dec 14, 2023
View reviewed changes
Copy link
Contributor

@ian0371 ian0371 left a comment

Choose a reason for hiding this comment

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

lgtm thanks!

@hyeonLewis hyeonLewis merged commit b01abb3 into klaytn:main Dec 14, 2023
2 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Dec 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@blukat29 blukat29 blukat29 approved these changes

@ian0371 ian0371 ian0371 approved these changes

@kjhman21 kjhman21 kjhman21 approved these changes

@aidan-kwon aidan-kwon aidan-kwon approved these changes

@neoofklaytn neoofklaytn Awaiting requested review from neoofklaytn neoofklaytn is a code owner

@scott-klaytn scott-klaytn Awaiting requested review from scott-klaytn scott-klaytn is a code owner

@hyunsooda hyunsooda Awaiting requested review from hyunsooda

Assignees

@hyeonLewis hyeonLewis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Discussion for Unified System Contract Management Process
8 participants
@hyeonLewis @Sotatek-TinnNguyen @blukat29 @ian0371 @hyunsooda @kjhman21 @aidan-kwon @scott-klaytn