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

Add Updates section to top level README #891

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

rphair
Copy link
Collaborator

@rphair rphair commented Aug 27, 2024

After consideration in #887 (comment) — for increased visibility in the community, especially to attract attention from stakeholders & implementors for any affected CIPs.

This uses a low bar for query compared to the Candidate list: only querying the Update tag without checking for a review State: tag... relying on that one bit of editor confirmation to keep spam or inappropriate update requests from being linked from the front page.

I don't see the harm in including likely "abandoned" or "deprecated" Update PR's here... in fact I believe it's important to include them because CIP/CPS updates aren't included in the Stalled / Waiting For Authors list (since the latter is only for new CIPs/CPSs).


(newly added section, rendered in branch)

@rphair rphair added the Bi-Weekly Notes / Editorial Housekeeping Publishing Bi-weekly meetings minutes / Mintor edits of public surrounding information label Aug 27, 2024
@rphair rphair added the State: Triage Applied to new PR afer editor cleanup on GitHub, pending CIP meeting introduction. label Aug 27, 2024
@rphair
Copy link
Collaborator Author

rphair commented Aug 30, 2024

@Crypto2099 this just needs to run through your review (no later than Tuesday's meeting) but I'll change the tag to Last Check since there are a bunch of new CIPs on the agenda.

@rphair rphair added State: Last Check Review favourable with disputes resolved; staged for merging. and removed State: Triage Applied to new PR afer editor cleanup on GitHub, pending CIP meeting introduction. labels Aug 30, 2024
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 b9377f8 into cardano-foundation:master Sep 3, 2024
@rphair rphair removed the State: Last Check Review favourable with disputes resolved; staged for merging. label Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bi-Weekly Notes / Editorial Housekeeping Publishing Bi-weekly meetings minutes / Mintor edits of public surrounding information
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants