Skip to content

Conversation

@avivkeller
Copy link
Member

@avivkeller avivkeller commented Jun 12, 2025

Per the vote, available at nodejs/collaborators-public-votes#17

See #6802

Copilot AI review requested due to automatic review settings June 12, 2025 21:49
@avivkeller avivkeller requested a review from a team as a code owner June 12, 2025 21:49
@vercel
Copy link

vercel bot commented Jun 12, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nodejs-org 🔄 Building (Inspect) Visit Preview Jun 12, 2025 9:49pm

Copy link
Contributor

Copilot AI left a 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 nodejs locked and limited conversation to collaborators Jun 12, 2025
@avivkeller
Copy link
Member Author

@nodejs/tsc

@MattIPv4
Copy link
Member

Do we have the ability to update the favicon to the pride variant as well?

@avivkeller avivkeller added fast-track Fast Tracking PRs github_actions:pull-request Trigger Pull Request Checks labels Jun 12, 2025
@github-actions github-actions bot removed the github_actions:pull-request Trigger Pull Request Checks label Jun 12, 2025
@avivkeller
Copy link
Member Author

Fast tracking, this can land once TSC approval

Copy link
Contributor

@bmuenzenmeyer bmuenzenmeyer left a 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 🏳️‍🌈

@avivkeller avivkeller added this pull request to the merge queue Jun 12, 2025
Merged via the queue into main with commit 41ed5ee Jun 12, 2025
18 of 19 checks passed
@avivkeller avivkeller deleted the avivkeller-patch-1 branch June 12, 2025 22:17
@ovflowd

This comment has been minimized.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

fast-track Fast Tracking PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants