Updated dependencies#362
Updated dependencies#362NullVoxPopuli merged 6 commits intoember-tooling:mainfrom ijlee2:update-dependencies
Conversation
There was a problem hiding this comment.
did you rebase? I'm surprised this diff is still here 🤔
There was a problem hiding this comment.
Hi, I did rebase after #363 had been merged.
I'm not sure what about the diff surprised you. It looks okay to me (i.e. the changes to this file are what I had intended). Can you clarify for me?
There was a problem hiding this comment.
sorry, I guess I'm surprisde there is still non-dependency updates in the dependency update PR.
since the refactor is related to content-tag, it should be in a separate PR, and maybe content-tag updating also be in a separate PR.
This PR is in conflict tho -- I pulled some of your changes out to another PR, #365
thanks for doing this! It's good to get things updated.
With #365 merged, this #362 will be a bit smaller, and we can keep iterating 🎉
There was a problem hiding this comment.
thanks! now this one has conflicts 🙈
| "eslint-plugin-n": "^17.12.0", | ||
| "eslint": "^9.28.0", | ||
| "eslint-config-prettier": "^10.1.5", | ||
| "eslint-plugin-jsdoc": "^50.8.0", |
There was a problem hiding this comment.
Note, v51.0.0 drops support Node 18.
https://github.com/gajus/eslint-plugin-jsdoc/releases/tag/v51.0.0
Background
A few dependencies have had a major release:
content-tageslintpnpmprettier(likely some breaking changes in a minor version)typescript-eslintvitestBy updating these to the latest version, we can continue to maintain this project.