Skip to content

Update node dependencies - #830

Merged
AdrianDAlessandro merged 12 commits into
mainfrom
node/update-node-dependencies
Nov 2, 2026
Merged

AdrianDAlessandro merged 12 commits into
mainfrom
node/update-node-dependencies

Conversation

@sbland

@sbland sbland commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Description

Updates node dependencies to latest version if possible. Currently includes:

  • Vitest
  • Storybook
  • Prettier
  • jsdom
  • rollup
  • eslint
  • stylelint

Type of change

  • Documentation (non-breaking change that adds or improves the documentation)
  • New feature (non-breaking change which adds functionality)
  • Optimization (non-breaking, back-end change that speeds up the code)
  • Technical work (non-breaking, change which is work as part of a new feature)
  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (whatever its nature)

Key checklist

  • All tests pass (eg. python -m pytest)
  • The documentation builds and looks OK (eg. mkdocs serve)
  • Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

@sbland
sbland force-pushed the node/update-node-dependencies branch from 8e4a066 to fef434f Compare October 28, 2026 12:41
@codecov

codecov Bot commented Oct 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sbland

sbland commented Oct 28, 2026

Copy link
Copy Markdown
Contributor Author

We may need to move from commonJS to ESM
This is blocking upgrading eslint and stylelint packages.

There are also some deprecated SCSS warnings.

@AdrianDAlessandro AdrianDAlessandro left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Happy with this! I've left one comment about the readme.

This needs to be rebased and have the package-lock.json to be fixed up before merging. Once you've done that, go ahead and merge it!

Comment thread node/README.md Resolved
@AdrianDAlessandro

Copy link
Copy Markdown
Collaborator

@sbland I made the fixes I suggested above. Got a request to make a release, and I wanted to get these changes in for that release

@AdrianDAlessandro
AdrianDAlessandro merged commit 587bf53 into main Nov 2, 2026
7 checks passed
@AdrianDAlessandro
AdrianDAlessandro deleted the node/update-node-dependencies branch November 2, 2026 15:19
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