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.

Sign up for GitHub

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

Jump to bottom

Chore upgrade docs #132

Merged
merged 14 commits into from
Dec 4, 2023
Merged

Chore upgrade docs #132

merged 14 commits into from
Dec 4, 2023

Conversation

josheleonard
Copy link
Contributor

Upgrades Docusaurus to version 3

@josheleonard josheleonard self-assigned this Nov 15, 2023
@socket-security Socket Security
Copy link

socket-security bot commented Nov 15, 2023 •
edited
Loading

New and updated dependencies detected. Learn more about Socket for GitHub ↗︎

Packages Version New capabilities Transitives Size Publisher
@docusaurus/module-type-aliases 3.0.0 None +37 12.6 MB slorber
@docusaurus/preset-classic 3.0.0 shell, environment +528 94.3 MB slorber
@docusaurus/types 3.0.0 None +35 12.6 MB slorber
@mdx-js/react 3.0.0 None +5 1.62 MB wooorm
@docusaurus/core 2.4.0...3.0.0 shell +475/-444 86.7 MB slorber
postcss 8.4.23...8.4.31 None +0/-0 197 kB ai
prism-react-renderer 1.3.5...2.1.0 None +1/-0 724 kB formidablelabs

@diracdeltas
Copy link
Member

Could you add yarn audit to the build-publish action or some other pre-merge action? so it's easy to see if new deps fail the audit

@josheleonard
Copy link
Contributor Author

@diracdeltas , added the audit step to the GH actions flow. The audit is failing now though

@josheleonard
Copy link
Contributor Author

@diracdeltas , The yarn audit findings are now fixed

@josheleonard
Copy link
Contributor Author

@diracdeltas , ok to merge this?

kdenhartog
kdenhartog approved these changes Dec 4, 2023
View reviewed changes
Copy link
Member

@kdenhartog kdenhartog left a comment

Choose a reason for hiding this comment

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

Lgtm

@josheleonard josheleonard merged commit 2e28385 into main Dec 4, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kdenhartog kdenhartog kdenhartog approved these changes

@bbondy bbondy Awaiting requested review from bbondy

@diracdeltas diracdeltas Awaiting requested review from diracdeltas

Assignees

@josheleonard josheleonard

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@josheleonard @diracdeltas @kdenhartog