-
Notifications
You must be signed in to change notification settings - Fork 213
chore: Bump all remaining deps, other than react-native/expo ones #2710
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
📝 No Changeset FoundThis PR doesn't include a changeset. A changeset (and the release label) is required to release a new version. How to add a changesetRun this command and follow the prompts: pnpm changesetRemember: Never use |
|
Size Change: -18.6 kB (-0.36%) Total Size: 5.09 MB
ℹ️ View Unchanged
|
This respects the minimum package age setting
Radu-Raicea
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The AI module looks fine, no major version updates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Additional Comments (1)
-
packages/browser/src/__tests__/autocapture-utils.test.ts, line 3 (link)syntax: inconsistent import style with
loader.test.tswhich usesimport * as sinon from 'sinon'- sinon 21.0.0 uses ESM exports
12 files reviewed, 1 comment
b603800 to
c4ef781
Compare
| - name: Ensure modern npm (OIDC support) | ||
| shell: bash | ||
| run: npm install -g [email protected] | ||
| run: sudo npm install -g [email protected] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yuck, really?!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This actually broke in a completely unrelated way, I'm not sure anything in my PR made this necessary or there was just an update to the runners
pauldambra
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like deps
Follow on from #2709
I've bumped some more pnpm deps, this time it's everything that shows up in
pnpm audit(excluding react native and expo) + runningpnpm updatein the browser and ai packageRelease info Sub-libraries affected
Libraries affected
Checklist
If releasing new changes
pnpm changesetto generate a changeset file