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

Update dev dependencies (to silence some warnings on some sites) #84

Merged
merged 5 commits into from
Jul 16, 2024

Conversation

voxpelli
Copy link
Contributor

As some tools show scary warnings despite it being dev dependencies that are outdated I updated them for you to get rid of it.

Also updated the GH Actions while I was at it, and since the updated dependencies officially need at least Node 18 to run, I dropped everything older.

Skärmavbild 2024-07-16 kl  16 54 09

(I actually found this during a Twitch stream where someone was talking about download counts of modules, and I found that this is the most downloaded one on npm, and then stumbled upon it on that site and saw the red warning and thought I would go here and see if you were aware of it, and since there were no issues or PR:s related to it I decided to make one. Please feel free to not merge as the error warnings are clearly bogus as this module has no non-dev dependencies)

@sindresorhus
Copy link
Member

As some tools show scary warnings despite it being dev dependencies that are outdated I updated them for you to get rid of it.

You should rather report the issue to such sites.

@voxpelli
Copy link
Contributor Author

Already done :)

@sindresorhus
Copy link
Member

I can merge this, but it will not make a difference since those sites check the published package, and I'm not going to publish a new major version just because those sites are flawed.

@sindresorhus sindresorhus merged commit 7e35811 into chalk:main Jul 16, 2024
3 checks passed
@voxpelli voxpelli deleted the voxpelli/update-dev-dependencies branch July 16, 2024 18:44
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.

2 participants