NO-ISSUE: Full dependencies update#91
Conversation
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates workspace dependency versions and adjusts package TypeScript compiler settings as part of a repo-wide dependency refresh.
Changes:
- Bumped multiple workspace catalog dependencies (Storybook, Vitest, React, Vite, TypeScript, jsdom, etc.).
- Simplified per-package
tsconfig.jsonby removingbaseUrland replacingrootDirswithrootDir.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| pnpm-workspace.yaml | Updates the shared pnpm catalog dependency versions used across packages. |
| packages/serverless-workflow-diagram-editor/tsconfig.json | Adjusts TS compiler options (removes baseUrl, sets rootDir). |
| packages/i18n/tsconfig.json | Adjusts TS compiler options (removes baseUrl, sets rootDir). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
There was a problem hiding this comment.
Thanks, LGTM, only one it doesnt cover is probably this one #66 but we can merge that separately
Signed-off-by: fantonangeli <fabrizio.antonangeli@gmail.com>
|
@lornakelly I would prefer to only update pnpm dependencies here and update the GHAs in a different PR or merge the dependabot one |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 8 out of 9 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Full dependencies update + tsconfigs update