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

Dependency cleanup #434

Merged
merged 3 commits into from
Dec 5, 2023
Merged

Dependency cleanup #434

merged 3 commits into from
Dec 5, 2023

Conversation

BrandtH22
Copy link
Collaborator

  • removes unused dependencies, updates other dependencies to latest versions
  • various formatting updates to align with latest dep3endencies
  • updates to the review and deploy to use node 16 and remove yarn

- removes excess, unused dependencies
- updates dependencies to their latest versions
updates the deploy method to use node18
- updating mdx to the latest version leads it to be more strict about formatting, this has various fixes to resolve those issues

- the deploy and review were using node 16 with review using yarn, this updates to node 18 and replaces yarn with npm
@danieljperry
Copy link
Contributor

It's working for me locally.

@BrandtH22 BrandtH22 merged commit 5c48f32 into main Dec 5, 2023
8 checks passed
@BrandtH22 BrandtH22 deleted the dependency-cleanup branch December 5, 2023 01:31
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