Skip to content

Conversation

@SBoudrias
Copy link
Contributor

Add 'overrides' property to package.json type definition.

Per https://docs.npmjs.com/cli/v9/configuring-npm/package-json#overrides

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Updated the comment for the 'overrides' property to clarify its purpose.
@SBoudrias
Copy link
Contributor Author

@som-sm addressed your review comments 🤞🏻

Copy link
Collaborator

@som-sm som-sm left a comment

Choose a reason for hiding this comment

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

LGTM, just add undefined as a valid value.

@SBoudrias
Copy link
Contributor Author

Updated!

@sindresorhus sindresorhus merged commit 8beaa89 into sindresorhus:main Nov 4, 2025
8 checks passed
@SBoudrias SBoudrias deleted the patch-2 branch November 4, 2025 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants