build(bicep-types-radius): bump tsp and deps#11565
build(bicep-types-radius): bump tsp and deps#11565DariuszPorowski wants to merge 2 commits intomainfrom
Conversation
Signed-off-by: Dariusz Porowski <3431813+DariuszPorowski@users.noreply.github.com>
Signed-off-by: Dariusz Porowski <3431813+DariuszPorowski@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR updates JavaScript toolchain versions for the repo and the hack/bicep-types-radius generator, aiming to keep development dependencies current.
Changes:
- Bump root
packageManagerpnpm version. - Update
hack/bicep-types-radius/src/generatordevDependencies, including TypeScript 6.x, ESLint, and yargs 18. - Regenerate the generator’s
pnpm-lock.yamlto reflect the updated dependency graph.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
package.json |
Updates the pinned pnpm version via packageManager. |
hack/bicep-types-radius/src/generator/package.json |
Bumps generator devDependencies (TS/ESLint/yargs/etc.). |
hack/bicep-types-radius/src/generator/pnpm-lock.yaml |
Lockfile updates resulting from the dependency bumps. |
Files not reviewed (1)
- hack/bicep-types-radius/src/generator/pnpm-lock.yaml: Language not supported
| "ts-node": "^10.9.2", | ||
| "typescript": "^5.6.3", | ||
| "yargs": "^17.7.2" | ||
| "typescript": "^6.0.2", | ||
| "yargs": "^18.0.0" |
There was a problem hiding this comment.
This bumps the generator to TypeScript 6.0.2 and yargs 18.0.0, but the generator entrypoint still uses the pre-TS6 yargs default-import pattern (import yargs from 'yargs' and then chaining without calling yargs(process.argv.slice(2))/hideBin). With TS6+yargs, this commonly breaks CJS/ESM interop at compile-time or runtime. Either update the generator CLI code/tsconfig to the yargs 18-compatible usage, or keep TypeScript/yargs pinned to the last known working major versions.
Radius functional test overviewClick here to see the test run details
Test Status⌛ Building Radius and pushing container images for functional tests... |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #11565 +/- ##
=======================================
Coverage 51.40% 51.40%
=======================================
Files 699 699
Lines 44156 44156
=======================================
Hits 22698 22698
Misses 19291 19291
Partials 2167 2167 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Description
This pull request updates several development dependencies and tools to their latest versions to ensure compatibility, security, and access to new features. The main changes are version bumps for dependencies in both the root
package.jsonand thehack/bicep-types-radius/src/generator/package.json.Type of change
Fixes: #11530
Contributor checklist
Please verify that the PR meets the following requirements, where applicable: