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

CIP-0100 | Move Acknowledgements into optional section #776

Merged

Conversation

rphair
Copy link
Collaborator

@rphair rphair commented Mar 6, 2024

This follows up #763 for other proposals identified as having lists of names before the CIP's or CPS's terms of documentation & Internet search significance.

@Quantumplation this covers your documents identified here on Discord (cc @klntsky) if skipping CPS-0007... which I would recommend because the link to the end section would be nearly as much text as the names currently there. If any objections (for the sake of consistency) we can update that one as well.

The remaining documents are these from @Ryun1 - one currently open PRs (so can be modified there):

and one other merged document which can be added here or "fixed" at the author's discretion in a separate PR:

p.s. @KtorZ it also corrects the spelling of your last name. 😇

@rphair rphair added the Update Adds content or significantly reworks an existing proposal label Mar 6, 2024
@rphair rphair requested review from Ryun1 and Crypto2099 March 6, 2024 13:39
Copy link
Collaborator

@Ryun1 Ryun1 left a comment

Choose a reason for hiding this comment

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

LGTM ✅

Copy link
Collaborator

@Crypto2099 Crypto2099 left a comment

Choose a reason for hiding this comment

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

LGTM

@rphair rphair merged commit 31aa220 into cardano-foundation:master Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Update Adds content or significantly reworks an existing proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants