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

February npm updates #11804

Merged
merged 3 commits into from
Jan 31, 2024
Merged

February npm updates #11804

merged 3 commits into from
Jan 31, 2024

Conversation

jjhembd
Copy link
Contributor

@jjhembd jjhembd commented Jan 31, 2024

Description

This PR updates outdated npm dependencies, excluding those which are pinned and that we have an issue tracking (see #10956).

Issue number and link

Resolves #10712.

Testing plan

Run npm install, then verify that the test.mjs script from #10684 runs without error.

Author checklist

  • I have submitted a Contributor License Agreement
  • I have added my name to CONTRIBUTORS.md
  • I have updated CHANGES.md with a short summary of my change
  • I have added or updated unit tests to ensure consistent code coverage
  • I have update the inline documentation, and included code examples where relevant
  • I have performed a self-review of my code
@jjhembd jjhembd requested a review from jjspace January 31, 2024 21:48
@jjhembd
Copy link
Contributor Author

jjhembd commented Jan 31, 2024

The new version of markdownlint-cli is a little stricter about tables and images in README files. I fixed a few things in CustomShaderGuide and FabricGuide to satisfy the new standard.

@ggetz
Copy link
Contributor

ggetz commented Jan 31, 2024

Awesome, thanks for the updates @jjhembd! Glad we can close #10712.

@ggetz ggetz merged commit b9da04d into main Jan 31, 2024
9 checks passed
@ggetz ggetz deleted the feb-package-updates branch January 31, 2024 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants