-
Notifications
You must be signed in to change notification settings - Fork 367
ci(repo): Version packages #6350
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 Git ↗︎
|
📝 WalkthroughWalkthroughThis update involves version increments and changelog additions for multiple Suggested labels
Suggested reviewers
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (51)
💤 Files with no reviewable changes (7)
✅ Files skipped from review due to trivial changes (12)
🚧 Files skipped from review as they are similar to previous changes (32)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
🧹 Nitpick comments (3)
packages/localizations/package.json (1)
3-3
: Consider adding a regression test for the newes-MX
localizationThe patch bump is fine, but the PR introduces translated content without any accompanying test to ensure the bundle still exports all expected keys.
A lightweight Jest/Vitest check that loadses-MX/index.ts
(or the compiled JS) and asserts the presence of a well-known phrase would guard against accidental breakage in future releases.packages/clerk-js/CHANGELOG.md (1)
3-9
: Add more context to the 5.74.2 changelog entryThe entry only states that
@clerk/localizations
was bumped. A short note explaining why (i.e., the updatedes-MX
translations from PR #5663) would make this log clearer and more consistent with the surrounding entries.
Example:## 5.74.2 ### Patch Changes - Bump `@clerk/localizations` to `3.20.2` to include the latest `es-MX` translation improvements (@LFCisneros, PR #5663).packages/chrome-extension/CHANGELOG.md (1)
7-8
: Correct malformed “Updated dependencies” bulletThe new entry deviates from all prior changelog lines by leaving an empty
[]
placeholder.
This breaks the consistently-used pattern and looks like a rendering glitch when viewed on GitHub.- - Updated dependencies []: + - Updated dependencies:(no hash is fine—earlier entries omit the hash entirely when none is available)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (9)
.changeset/stale-signs-cheer.md
(0 hunks)packages/chrome-extension/CHANGELOG.md
(1 hunks)packages/chrome-extension/package.json
(1 hunks)packages/clerk-js/CHANGELOG.md
(1 hunks)packages/clerk-js/package.json
(1 hunks)packages/expo/CHANGELOG.md
(1 hunks)packages/expo/package.json
(1 hunks)packages/localizations/CHANGELOG.md
(1 hunks)packages/localizations/package.json
(1 hunks)
💤 Files with no reviewable changes (1)
- .changeset/stale-signs-cheer.md
🧰 Additional context used
📓 Path-based instructions (5)
**/*.{js,jsx,ts,tsx,json,css,scss,md,yaml,yml}
Instructions used from:
Sources:
📄 CodeRabbit Inference Engine
- .cursor/rules/development.mdc
packages/*/package.json
Instructions used from:
Sources:
📄 CodeRabbit Inference Engine
- .cursor/rules/global.mdc
- .cursor/rules/monorepo.mdc
**/*
Instructions used from:
Sources:
⚙️ CodeRabbit Configuration File
packages/localizations/**/*
Instructions used from:
Sources:
📄 CodeRabbit Inference Engine
- .cursor/rules/monorepo.mdc
packages/**/CHANGELOG.md
Instructions used from:
Sources:
📄 CodeRabbit Inference Engine
- .cursor/rules/development.mdc
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: Build Packages
- GitHub Check: Formatting | Dedupe | Changeset
- GitHub Check: Analyze (javascript-typescript)
🔇 Additional comments (5)
packages/expo/package.json (1)
3-3
: Patch version bump looks good – double-check peer compatibility
2.14.10
correctly tracks the dependency bumps, but please verify (CI or manual smoke-test) that Expo projects consuming this build resolve@clerk/[email protected]
without yarn/npm hoisting surprises.
If peer warnings surface, a follow-up changeset might be required.packages/chrome-extension/package.json (1)
3-3
: Patch version bump is semver-correctNo issues spotted with the update from
2.5.10
→2.5.11
; change aligns with a patch-level release.packages/clerk-js/package.json (1)
3-3
: Version bump LGTMPatch increment
5.74.1 → 5.74.2
is appropriate for the localization update; no other issues spotted.packages/expo/CHANGELOG.md (1)
3-10
: Missing commit references in the “Updated dependencies” bulletEvery previous entry includes commit hashes inside back-links (e.g.
[abc123]
) so that readers can trace the dependency bump.
This entry renders asUpdated dependencies []:
which is an empty link and breaks the convention.Diff suggestion:
-### Patch Changes - -- Updated dependencies []: +### Patch Changes + +- Updated dependencies [[`<commit>`]]: <!-- replace with actual short-SHA -->Please add the corresponding commit SHA(s) (or drop the square-brackets block if none are available) to keep the changelog consistent and navigable.
[ suggest_nitpick ]packages/localizations/CHANGELOG.md (1)
3-8
: Changelog entry LGTMThe new 3.20.2 patch section is concise, follows the existing format, and clearly credits the contributor and PR reference. No further action required.
@clerk/agent-toolkit
@clerk/astro
@clerk/backend
@clerk/chrome-extension
@clerk/clerk-js
@clerk/dev-cli
@clerk/elements
@clerk/clerk-expo
@clerk/expo-passkeys
@clerk/express
@clerk/fastify
@clerk/localizations
@clerk/nextjs
@clerk/nuxt
@clerk/clerk-react
@clerk/react-router
@clerk/remix
@clerk/shared
@clerk/tanstack-react-start
@clerk/testing
@clerk/themes
@clerk/types
@clerk/upgrade
@clerk/vue
commit: |
371483a
to
057df07
Compare
057df07
to
3895e5d
Compare
3895e5d
to
777dc16
Compare
777dc16
to
af554f0
Compare
af554f0
to
9ae9c54
Compare
9ae9c54
to
d851888
Compare
d851888
to
693533f
Compare
693533f
to
1ca09d7
Compare
1ca09d7
to
e74e470
Compare
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.
Releases
@clerk/[email protected]
Minor Changes
payerType[]
withforPayerType
typed as'org' | 'user'
. (#6342) by @panteliselefPatch Changes
8ff2aef
,6c5cd88
,80abb5d
,4c3d6c1
,438e499
,bbfaca8
,35da3e8
,10e1060
,6080456
,bd84c62
,305e5e4
]:@clerk/[email protected]
Minor Changes
payerType[]
withforPayerType
typed as'org' | 'user'
. (#6342) by @panteliselef@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
Refine Traditional Chinese (
zh-TW
) translations (#5683) by @anilahsuAdd Persian (
fa-IR
) language (#6063) by @XerxesCoderExtend
ru-RU
localization (#5698) by @ZharaskhanAmanfeat(localizations): Update
sk-SK
localization (#5497) by @radbleskAdd
sr-RS
to the list of exported and available localizations (#6352) by @tmilewskifeat(localization): Add Hindi, Bengali, Tamil, Telugu, and Malay language support (#5534) by @vanikya
Add missing French locales to support new Clerk's Billing feature (#5944) by @bde-maze
chore(localizations): Align
en-US
strings for danger zone with other translations (#5800) by @joschiAdd Kazakh (kk-KZ) language (#5684) by @azekowka
Update
es-MX
translations (#5663) by @LFCisnerosUpdated dependencies [
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:@clerk/[email protected]
Patch Changes
10e1060
]:Summary by CodeRabbit
New Features
Chores
Documentation