Skip to content
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

Temporarily add maintainers to package.json to handle NPM schema change #2080

Merged

Conversation

PatrickStankard
Copy link
Contributor

♻️ Current situation

NPM stopped returning the maintainers key from the registry when viewing a versioned package. Is this intentional? Nobody knows. I have a support ticket open with NPM since last Thursday, and the thread on GitHub Communities hasn't seen a response. This change has prevented the Homebridge UI from showing a plugin's author, as well as showing users that there is a Homebridge UI update from within Homebridge.

💡 Proposed solution

Unfortunately, users will not see that there are Homebridge UI updates until maintainers is manually added to package.json. This key is supposed to be set by NPM themselves, but it appears that you can set it yourself. I copied the maintainers from the top-level package metadata here. This change can eventually be reverted if NPM starts setting maintainers in the versioned package metadata again, or if #2079 is merged, and some amount of time has passed to ensure users have updated their Homebridge UI.

@github-actions github-actions bot added the latest Related to Latest Branch label Jun 25, 2024
@bwp91 bwp91 merged commit b064018 into homebridge:latest Jun 25, 2024
3 checks passed
donavanbecker added a commit that referenced this pull request Jun 25, 2024
## v4.56.4 (2024-06-25)

### Other Changes

- Temporarily add maintainers to package.json to handle NPM schema change (#2080) (@PatrickStankard)
@PatrickStankard PatrickStankard deleted the set-maintainers-in-package-json branch June 25, 2024 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
latest Related to Latest Branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants