Skip to content

chore(deps): updated all dependencies (pre-req for #207) - #208

Closed
hildjj wants to merge 2 commits into
nearform:masterfrom
hildjj:update-dependencies
Closed

chore(deps): updated all dependencies (pre-req for #207)#208
hildjj wants to merge 2 commits into
nearform:masterfrom
hildjj:update-dependencies

Conversation

@hildjj

@hildjj hildjj commented Aug 17, 2025

Copy link
Copy Markdown
Contributor

Except left eslint at 8.57 since moving to eslint
9 requires maintainer decisions. Added @types/node to get tests to pass. Refreshed package-lock.json to ensure that "npm audit" is clean.

Pre-requisite to fixing #207

Except left eslint at 8.57 since moving to eslint
9 requires maintainer decisions.  Added @types/node
to get tests to pass.  Refreshed package-lock.json
to ensure that "npm audit" is clean.
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Aug 17, 2025
@github-actions

Copy link
Copy Markdown

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

@dosubot dosubot Bot added the dependencies Pull requests that update a dependency file label Aug 17, 2025
@hildjj hildjj changed the title chore(deps): updated all dependencies chore(deps): updated all dependencies (pre-req for #207) Aug 17, 2025
@hildjj

hildjj commented Aug 17, 2025

Copy link
Copy Markdown
Contributor Author

I don't know what the bot wants me to change. Please feel free to edit the descriptions as necessary.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Aug 18, 2025
@simoneb

simoneb commented Aug 18, 2025

Copy link
Copy Markdown
Member

CI is failing. Note that we usually prefer relying on dependabot for the dependency bumps.

@hildjj

hildjj commented Aug 18, 2025

Copy link
Copy Markdown
Contributor Author

Let's see if this works; npm ci runs clean on my machine now.

@hildjj

hildjj commented Aug 18, 2025

Copy link
Copy Markdown
Contributor Author

Also, I'm happy to let this PR go, and rebase #209 wherever you like.

@simoneb

simoneb commented Aug 19, 2025

Copy link
Copy Markdown
Member

CI is still failing

@simoneb

simoneb commented Aug 19, 2025

Copy link
Copy Markdown
Member

@hildjj one Q: why is this a prerequisite for #209 ?

@hildjj

hildjj commented Aug 20, 2025

Copy link
Copy Markdown
Contributor Author

CI fails on the master branch also for node 24.6, but not for node 24.5 with the same error. I'm looking into why.

@hildjj

hildjj commented Aug 20, 2025

Copy link
Copy Markdown
Contributor Author

This CI error is caused by nodejs/node#58253 which changed Assert to become a class.

  • I'm going to close this PR as it's clear you want to manage dependencies otherwise.
  • I'll start a new PR where we can discuss the fix for this (my approach is going to be to add another sed to compare.sh, to ensure the tests work on all supported versions of node).
  • I'll rebase fix #207: assign diagnostic to correct test #209 on top of master, with no changes to dependencies.

@hildjj hildjj closed this Aug 20, 2025
@hildjj
hildjj deleted the update-dependencies branch August 20, 2025 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants