-
-
Notifications
You must be signed in to change notification settings - Fork 23k
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
Theme: add new some themes #3138
Conversation
Theme: add new some themes
@IRMABaitussalam is attempting to deploy a commit to the github readme stats Team on Vercel. A member of the Team first needs to authorize it. |
Automated Theme PreviewHi, thanks for the theme contribution. Please read our theme contribution guidelines. Warning Keep in mind that we already have a vast collection of different themes. To keep their number manageable, we began to add only themes supported by the community. Your pull request with theme addition will be merged once we get enough positive feedback from the community in the form of thumbs up (see #1935). Remember that you can also support themes of other contributors that you liked to speed up their merge. Note Also, note that if this theme is exclusively for your personal use, then instead of adding it to our theme collection, you can use card customization options. ✔️ Theme PR does adhere to our guidelines. Test results
Result: ✔️ All themes are valid. DetailsQamariyah theme previewtitle_color: Darkes_cadillac theme previewtitle_color: Darkes_mercury theme previewtitle_color: Darkes_sun theme previewtitle_color: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some themes are invalid. See the Automated Theme Preview comment above for more information.
@IRMABaitussalam, although I love the themes, I think it is a bit hard for us to review the popularity of your themes when you included so many in your PRs. @qwerty541, @anuraghazra, what do you think should update the theme preview CI to display a warning when multiple themes are added or maybe let it add a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@IRMABaitussalam, although I love the themes, I think it is a bit hard for us to review the popularity of your themes when you included so many in your PRs.
@qwerty541, @anuraghazra, what do you think should update the theme preview CI to display a warning when multiple themes are added or maybe let it add a
batch
label? I initially added the option for multiple themes to allow people to supply both a dark an light version of a theme 🤔.
@rickstaa I think that we can update our themes contribution guidelines wtih adding a new rule - one theme per pull request. May be sutiations when user wants to add some versions of same theme can be exceptional. I hope it will be enough for now. If users continue adding pull requests with batch of themes we can update our theme review script to request changes instead of approving in such cases. I can open pull request with changes into documentation if you are up to merge.
That sounds like a good start! How should we deal with dark and light themes? Do we still allow, for example, |
I think that sutiations when user wants to add some versions of same theme can be exceptional.
If users continue adding pull requests with batch of themes after changes in documentation we can update our theme review script to request changes instead of approving in such cases. |
We are currently pausing addition of new themes. If this theme is exclusively for your personal use, then instead of adding it to our theme collection, you can use card customization options. |
No description provided.