-
Notifications
You must be signed in to change notification settings - Fork 3
Closed
Description
The project's NPM dependencies are outdated, and Renovate updated them. But, due to breaking changes, manual fixes are necessary; if you update all packages even to at most minor versions, you'll be greeted with the following warnings and errors in VS, necessitating fixes in other projects too:
- Open an issue branch from the branch of Update All packages #114 and once you're done with that, merge the branch of Update All packages (major) #115 too.
- Check all updated dependencies: Are they actually all needed?
- Fix any compatibility issues, including in other OSOCE projects, like the Base Theme as seen in the above screenshot.
- To fix any "require() of ES Module ... not supported" type error, you can use
await import()inside an asynchronous method, like here. - When upgrading ESLint to the latest v9.x then configurations have to be migrated, see here.
- When upgrading StyleLint "The CommonJS Node.js API is deprecated."
- To fix any "require() of ES Module ... not supported" type error, you can use
- Should we pin the PNPM version in every project's
"packageManager"property? It seems like a safe way to take dependency on a given PNPM version, which can also be auto-updated by Renovate. If indeed, thencorepack prepare pnpmcan be removed from NodeJsExtensionsNpmPackage.targets, theenable-corepackaction, and documented (and ideally automatically enforced somehow) that this property should be always added. - Remove the temporary "All packages" group from renovate.json5 here and "ESLint" group from Lombiq.VueJs.
After merging the issue branch, both renovate PRs should close itself.
Metadata
Metadata
Assignees
Labels
No labels
