Skip to content

Update index.js #4203

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update index.js #4203

wants to merge 2 commits into from

Conversation

yixuanwu4
Copy link

Hi :)

I updated index.js with a new theme based on the colors of the Pokémon Lucario.

Cheers!

Copy link

vercel bot commented Apr 2, 2025

@yixuanwu4 is attempting to deploy a commit to the github readme stats Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the themes Feature, Enhancement, Fixes related to themes. label Apr 2, 2025
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Something went wrong in the theme preview action: PR diff is not a valid theme JSON object.

@github-actions github-actions bot added the invalid The bug/issue caused by minor mistakes. label Apr 2, 2025
Co-authored-by: Daniel Ângelo <84107769+danielangelo1@users.noreply.github.com>
Copy link
Contributor

github-actions bot commented Apr 7, 2025

Automated Theme Preview

Hi, 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 👍 emojis (see #1935). We expect to see at least 10-15 thumbs up before making a decision to merge your pull request into the master branch. Remember that you can also support themes of other contributors that you liked to speed up their merge.

Warning

Please do not submit a pull request with a batch of themes, since it will be hard to judge how the community will react to each of them. We will only merge one theme per pull request. If you have several themes, please submit a separate pull request for each of them. Situations when you have several versions of the same theme (e.g. light and dark) are an exception to this rule.

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

  • ✔️ lucario

Result: ✔️ All themes are valid.

Details

Lucario theme preview

title_color: #4aa5d4 | icon_color: #da6771 | text_color: #fffd7d | bg_color: #1b2333

Preview Link

@github-actions github-actions bot removed the invalid The bug/issue caused by minor mistakes. label Apr 7, 2025
@yixuanwu4 yixuanwu4 requested a review from danielangelo1 April 7, 2025 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
themes Feature, Enhancement, Fixes related to themes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants