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

chore: upgrade devDependencies around docs site to latest #5071

Conversation

JoshuaKGoldberg
Copy link
Member

@JoshuaKGoldberg JoshuaKGoldberg commented Dec 28, 2023

PR Checklist

Overview

I upgraded many website-related packages to latest, then ran npm audit fix --force.

Includes changes from #5069. Leaving in draft until that PR is approved, after which I'll merge those changes into this branch and un-draft. I'm sending now as a reference.

There are a few vulnerabilities from old packages left after this. See #5070 (comment).

@coveralls
Copy link

Coverage Status

coverage: 94.337%. remained the same
when pulling b970f7d on JoshuaKGoldberg:docs-deps-and-npm-audit-fix
into 645469e on mochajs:master.

@@ -130,13 +129,14 @@
"rollup-plugin-visualizer": "^5.6.0",
"sinon": "^9.0.3",
"strip-ansi": "^6.0.0",
"svgo": "^1.3.2",
"svgo": "^3.1.0",
"taffydb": "^2.7.3",
Copy link
Member Author

@JoshuaKGoldberg JoshuaKGoldberg Dec 28, 2023

Choose a reason for hiding this comment

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

This is required by @mocha/docdash. It's not listed in the package's dependencies but is required by a file. Once #5072 is resolved, we shouldn't need this explicit dep anymore.

@JoshuaKGoldberg JoshuaKGoldberg removed the status: blocked Waiting for something else to be resolved label Jan 30, 2024
@JoshuaKGoldberg JoshuaKGoldberg added the status: waiting for author waiting on response from OP - more information needed label Mar 4, 2024
@voxpelli
Copy link
Member

voxpelli commented Aug 5, 2024

This one needs to be updated now, but preferably it would wait until #5042 is merged

@voxpelli
Copy link
Member

#5042 is now merged :)

@JoshuaKGoldberg
Copy link
Member Author

Per #5207, we're going to go with a website overhaul instead.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting for author waiting on response from OP - more information needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants