-
Notifications
You must be signed in to change notification settings - Fork 775
Update minor dependencies #650
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
Update minor dependencies #650
Conversation
Are package.json and package-lock.json not in sync in this PR? I think if you just delete pacakge-lock.json and do |
f880332
to
b912b4a
Compare
b912b4a
to
d019376
Compare
No they weren't. I thought I'd keep the changes simple for review and history, and previous updates hadn't been changing
Not really. Since it doesn't matter technically and is just a nicety when reading I revised and squashed. Let me know if this isn't what you had in mind. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build, test, run locally all good. LGTM! 👍
Update packages to the latest versions permitted by their package specifier; no specifier changes. In most cases, these are minor version updates.
The exception is
prettier
, which needs a specifier update because it was added with a single version specifier in af16a53. There are no changes to code formatting with this update.Motivation and Context
Lessen the burden of future maintenance by keeping the project relatively up to date. Tracking minors version ought to make it easier to do major version upgrades.
How Has This Been Tested?
npm ci && npm run prettier-fix && npm run lint && npm run build && npm run test && npm run test:e2e
npx . npx @modelcontextprotocol/servers-everything
.Breaking Changes
No
Types of changes
Checklist
Additional context
Regarding
prettier
, I reviewedgit log -p -G prettier -- package.json
, but didn't see why it might have be pinned to3.3.3
.Unfortunately, these updates do not address the deprecating warnings emitted on install, which was my initial motivation. They're due to an old version of
test-include
being used bybabel
andbabel-plugin-istanbul
, and will be fixed when this PR lands there.@cliffhall 👋 Recently we discussed syncing
package.json
withpackage-lock.json
. I could do that as a separate PR?