-
Notifications
You must be signed in to change notification settings - Fork 24
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
Proposal: Add Emojis to the CEPS #3
Comments
Hmm, I'd be worried about emoji in commit messages personally since it makes the commit log messy in the git log output and GitHub history view. Since commit messages are forever I'd keep it simple with those for now. There is also a bit of an issue for people that don't have a font with Emoji support installed for their command line (e.g. older Windows). Alternatively, would having a listing of the current CEPs in the README with emojis help with the overview problem? E.g. a simple Markdown table like this:
BTW, here'd be my proposal for the set of emojis: 🚧 for Draft |
Good points. Yeah maybe having it in the readme would work. I think it would be just very easy to then see which ones someone could maybe look at if they are wanting to, and its part of the flow. We could use labels too, but I don't think they work as well due to needing to work just with a github function and is outside the actual code/files |
I like the idea of this information being in the Readme and kept up to date. |
Self writing READMEs are nice for this. There are some examples in those fancy profile page READMEs. I can work on that if needed. |
I agree that this should be implemented as a self writing README, otherwise it will absolutely become a pain to maintain. Is the idea to create a scheduled GitHub action that regenerates the index at some interval? Or is it preferred to use a bot/action that regenerates the index as part of the PR? A few other options for the deferred icon: |
Since each CEP is a single file, could use a emoji to signify its status? You can add these to the front of the commit message/PR. This would allow us to quickly see at a glance what the status of things are. I tried an example in the CEP-4 branch
🔨 for Draft
🛠️ Proposed
✔️ Accepted
➖ Rejected
✅ Implemented
⚡ Deferred (Not sure on this one)
@jezdez what do you think?
The text was updated successfully, but these errors were encountered: