-
-
Notifications
You must be signed in to change notification settings - Fork 29
chore: release package(s) #454
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
Conversation
WalkthroughThis update removes a changeset file documenting a patch and adds new changelog entries and version bumps for five Prettier plugin packages. The changes reflect a fix that removes a buggy Changes
Possibly related PRs
Suggested reviewers
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (11)
💤 Files with no reviewable changes (1)
✅ Files skipped from review due to trivial changes (3)
🚧 Files skipped from review as they are similar to previous changes (7)
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 (
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
prettier-plugin-autocorrect
prettier-plugin-pkg
prettier-plugin-sh
prettier-plugin-sql
prettier-plugin-toml
commit: |
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.
Important
Looks good to me! 👍
Reviewed everything up to 39183ac in 50 seconds. Click for details.
- Reviewed
155
lines of code in11
files - Skipped
0
files when reviewing. - Skipped posting
10
draft comments. View those below. - Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. packages/toml/CHANGELOG.md:39
- Draft comment:
Suggestion: Consider changing 'efficienter' to 'more efficient' for clarity. - Reason this comment was not posted:
Confidence changes required:33%
<= threshold50%
None
2. packages/sh/package.json:11
- Draft comment:
Verify if 'engines.node' set to '>=16.0.0' is intentional compared to other packages which support Node ^14.18.0. Ensure consistency if needed. - Reason this comment was not posted:
Confidence changes required:50%
<= threshold50%
None
3. .changeset/little-baboons-ring.md:1
- Draft comment:
The changeset file 'little-baboons-ring.md' is being removed. Confirm that this outdated changeset is no longer needed. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
4. packages/pkg/package.json:10
- Draft comment:
Notice the license is set to 'MPL-2.0', which differs from the other packages using MIT. Verify that this discrepancy is intentional. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
5. packages/toml/package.json:8
- Draft comment:
The 'author' field is 'Ray' while the 'maintainers' array lists 'JounQin'. Confirm that the division of roles is correct. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
6. packages/pkg/CHANGELOG.md:264
- Draft comment:
Typo found: 'badage' should be corrected to 'badge' to maintain consistency in documentation. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
7. packages/sh/package.json:5
- Draft comment:
In the description (line 5), consider revising 'also support simple format of ...' to something like 'also supports simple formatting of ...' for improved grammatical correctness. This is a minor, cosmetic improvement. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%
<= threshold50%
This comment is purely about grammatical correctness in the PR description, which is not relevant to the code changes themselves. It violates the rule against asking the PR author to update the PR description.
8. packages/sql/CHANGELOG.md:19
- Draft comment:
The commit message on line 19 uses 'jsox' in lowercase, while elsewhere in the changelog it's written as 'JSOX' (e.g., in the features for stringified params). Consider standardizing the capitalization for consistency. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
9. packages/sql/package.json:41
- Draft comment:
The keyword "postgre-sql" on line 41 might be a typographical error. Typically, it is spelled as "postgresql". Please double-check if this is the intended term. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
10. packages/toml/CHANGELOG.md:39
- Draft comment:
Typographical error: In line 39, 'wrapps' should be 'wraps' and 'efficienter' should be corrected to 'more efficient' or 'efficient'. - Reason this comment was not posted:
Comment was not on a location in the diff, so it can't be submitted as a review comment.
Workflow ID: wflow_xNvF3avfwouGg8iW
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
size-limit report 📦
|
📊 Package size report 0.03%↑
Unchanged files
🤖 This report was automatically generated by pkg-size-action |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #454 +/- ##
=======================================
Coverage 85.71% 85.71%
=======================================
Files 10 10
Lines 182 182
Branches 47 47
=======================================
Hits 156 156
Misses 25 25
Partials 1 1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (11)
.changeset/little-baboons-ring.md
(0 hunks)packages/autocorrect/CHANGELOG.md
(1 hunks)packages/autocorrect/package.json
(1 hunks)packages/pkg/CHANGELOG.md
(1 hunks)packages/pkg/package.json
(1 hunks)packages/sh/CHANGELOG.md
(1 hunks)packages/sh/package.json
(1 hunks)packages/sql/CHANGELOG.md
(1 hunks)packages/sql/package.json
(1 hunks)packages/toml/CHANGELOG.md
(1 hunks)packages/toml/package.json
(1 hunks)
💤 Files with no reviewable changes (1)
- .changeset/little-baboons-ring.md
⏰ Context from checks skipped due to timeout of 90000ms (4)
- GitHub Check: Codacy Static Code Analysis
- GitHub Check: Lint and Test with Node.js 22 on windows-latest
- GitHub Check: Lint and Test with Node.js 20 on windows-latest
- GitHub Check: Lint and Test with Node.js 18 on windows-latest
🔇 Additional comments (10)
packages/sh/package.json (1)
3-3
: Confirm version bump to 0.17.3
Theversion
field has been correctly updated to align with the coordinated patch release removing the buggymodule-sync
exports field.packages/sh/CHANGELOG.md (2)
3-4
: Document new release header
Added the## 0.17.3
section to introduce the patch release.
7-7
: Log patch changes correctly
The entry references PR #453 and commitb44651a
, accurately noting the removal of the buggymodule-sync
exports field.packages/sql/package.json (1)
3-3
: Confirm version bump to 0.19.1
The version update matches the new patch release and aligns with the changelog entry. No other fields were modified.packages/sql/CHANGELOG.md (1)
3-8
: Approve changelog entry for v0.19.1
The new section correctly documents the removal of the buggymodule-sync
exports field, references PR #453 and commitb44651a
, and follows the established formatting.packages/autocorrect/package.json (1)
3-3
: Version bump is accurate and consistent.
Theversion
field has been correctly updated from0.4.2
to0.4.3
to reflect the patch release. This aligns with the changelog entry and no other fields were unintentionally modified.packages/autocorrect/CHANGELOG.md (1)
3-8
: Changelog entry correctly documents the patch fix.
The new## 0.4.3
section includes the PR link#453
and commitb44651a…
, clearly describing the removal of the buggymodule-sync
exports field. The formatting mirrors previous entries and stays in sync withpackage.json
.packages/pkg/package.json (1)
3-3
: Version bump correctly updated to 0.19.1.
The package version has been incremented from 0.19.0 to 0.19.1 to reflect the patch release fixing themodule-sync
exports field.packages/pkg/CHANGELOG.md (1)
3-7
: Changelog entry for 0.19.1 is well-formatted and accurate.
The new “## 0.19.1” section clearly documents the patch change, references PR #453 and commitb44651a
, and matches the style of earlier entries.packages/toml/CHANGELOG.md (1)
3-8
: Verified new changelog entry for v2.0.5
The added section accurately follows the existing format, references PR #453 and commitb44651a
, and clearly describes the fix for themodule-sync
exports field.
39183ac
to
b7985b5
Compare
b7985b5
to
7082a3b
Compare
7082a3b
to
13015ea
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 master, this PR will be updated.
Releases
[email protected]
Patch Changes
b44651a
Thanks @JounQin! - fix: remove buggymodule-sync
exports field[email protected]
Patch Changes
b44651a
Thanks @JounQin! - fix: remove buggymodule-sync
exports field[email protected]
Patch Changes
b44651a
Thanks @JounQin! - fix: remove buggymodule-sync
exports field[email protected]
Patch Changes
b44651a
Thanks @JounQin! - fix: remove buggymodule-sync
exports field[email protected]
Patch Changes
b44651a
Thanks @JounQin! - fix: remove buggymodule-sync
exports fieldSummary by CodeRabbit