-
-
Notifications
You must be signed in to change notification settings - Fork 6.4k
chore(🏳️🌈): logo #7861
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
chore(🏳️🌈): logo #7861
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Pull Request Overview
This PR updates the site's configuration by adding a new logo variant property that reflects a pride theme.
- Added a new "logoVariant" property with the value "pride" in the configuration file.
Comments suppressed due to low confidence (1)
apps/site/site.json:6
- [nitpick] Consider adding comments or documentation to explain the usage and expected values for the new 'logoVariant' property, ensuring that maintainers and other developers understand the context of this change.
"logoVariant": "pride",
@nodejs/tsc |
Do we have the ability to update the favicon to the pride variant as well? |
Fast tracking, this can land once TSC approval |
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.
you beat me to it 🏳️🌈
Could you please attach the relevant issue on the PR description? |
Per the vote, available at nodejs/collaborators-public-votes#17
See #6802