-
-
Notifications
You must be signed in to change notification settings - Fork 69
chore: bump unrs-resolver
to v1.6.0
#433
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
🦋 Changeset detectedLatest commit: df43ad3 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
Pull Request Overview
This PR bumps the version of unrs-resolver to v1.6.0 while updating related documentation and configuration files. Key changes include:
- Updating the README.md to reorganize the Sponsors and Backers sections.
- Changing the import path in .nano-staged.js to use the new nano-staged module.
- Adding a changeset file documenting the version bump.
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.
File | Description |
---|---|
README.md | Updated sponsors section headers for clarity. |
.nano-staged.js | Modified import path to the updated module. |
.changeset/serious-cheetahs-wink.md | Added changeset for version bump. |
Files not reviewed (1)
- package.json: Language not supported
WalkthroughThis update primarily consists of dependency version bumps and minor documentation or formatting changes. The Changes
Possibly related PRs
Suggested reviewers
Poem
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
yarn install v1.22.22 📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (9)
⏰ Context from checks skipped due to timeout of 90000ms (6)
🔇 Additional comments (11)
✨ Finishing Touches
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:
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. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #433 +/- ##
=======================================
Coverage 63.58% 63.58%
=======================================
Files 6 6
Lines 368 368
Branches 48 48
=======================================
Hits 234 234
Misses 134 134 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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 63ea8ea in 36 seconds. Click for details.
- Reviewed
106
lines of code in4
files - Skipped
1
files when reviewing. - Skipped posting
12
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. .changeset/serious-cheetahs-wink.md:5
- Draft comment:
Changelog entry is clear. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
2. .nano-staged.js:1
- Draft comment:
Ensure renaming from lint-staged to nano-staged is intentional and consistent. - Reason this comment was not posted:
Confidence changes required:10%
<= threshold50%
None
3. README.md:331
- Draft comment:
Sponsors/backers section reorganization improves clarity. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
4. package.json:84
- Draft comment:
Dependency 'tinyglobby' version bump is noted; verify compatibility. - Reason this comment was not posted:
Confidence changes required:10%
<= threshold50%
None
5. package.json:85
- Draft comment:
Bumped 'unrs-resolver' to v1.6.0 as intended. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
6. package.json:88
- Draft comment:
Update to '@1stg/common-config' should be checked for breaking changes. - Reason this comment was not posted:
Confidence changes required:20%
<= threshold50%
None
7. package.json:111
- Draft comment:
Verify 'react' in devDependencies is intentional for this project. - Reason this comment was not posted:
Confidence changes required:30%
<= threshold50%
None
8. package.json:121
- Draft comment:
Removal of 'es5-ext' resolution seems intentional; confirm no downstream issues. - Reason this comment was not posted:
Confidence changes required:20%
<= threshold50%
None
9. .changeset/serious-cheetahs-wink.md:5
- Draft comment:
Changeset file looks good; bump message is clear. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
10. .nano-staged.js:1
- Draft comment:
Updated to use '@1stg/nano-staged/tsc'; this tooling rename is applied correctly. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
11. README.md:331
- Draft comment:
Sponsors & Backers section has been restructured for clarity; formatting improvements look good. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
12. package.json:84
- Draft comment:
Dependency version bumps and the switch from 'lint-staged' to 'nano-staged' are consistent; ensure all tooling is updated accordingly. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
Workflow ID: wflow_ZxDIlzm3lYzo2bBD
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
size-limit report 📦
|
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 26b8583 in 46 seconds. Click for details.
- Reviewed
192
lines of code in9
files - Skipped
0
files when reviewing. - Skipped posting
15
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. .changeset/serious-cheetahs-wink.md:2
- Draft comment:
Changeset looks correct for a version bump; ensure consistency with package.json. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
2. .nano-staged.js:1
- Draft comment:
Import path updated to '@1stg/nano-staged/tsc'; verify package renaming is intentional. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
3. README.md:42
- Draft comment:
Sponsors and Backers section has been reorganized; double-check all links and image URLs. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
4. package.json:84
- Draft comment:
Dependency versions (e.g. unrs-resolver and tinyglobby) have been bumped; confirm these updates are compatible. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
5. src/types.ts:5
- Draft comment:
Formatting improvements in inline comments; looks clearer now. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
6. .changeset/serious-cheetahs-wink.md:5
- Draft comment:
Changeset entry is clear; bump of unrs-resolver to v1.6.0 is correctly noted. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
7. .nano-staged.js:1
- Draft comment:
Renamed config to nano-staged and updated import; ensure CI and documentation reference the new name. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
8. README.md:332
- Draft comment:
Sponsors and Backers section has been restructured for clarity. Verify that the combined heading with nested subheadings aligns with project branding. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
9. eslint.config.js:7
- Draft comment:
JSDoc formatting update (@import spacing) improves clarity; no functional changes. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
10. package.json:81
- Draft comment:
Dependency versions have been bumped (e.g., unrs-resolver to v1.6.0, tinyglobby to v0.2.13). Ensure these updates are compatible and that any breaking changes are addressed. - Reason this comment was not posted:
Comment did not seem useful. Confidence is useful =0%
<= threshold50%
This comment is about dependency version changes and asks the author to ensure compatibility and address breaking changes. According to the rules, comments on dependency changes or asking the author to ensure compatibility are not allowed.
11. src/constants.ts:17
- Draft comment:
Reformatted doc comment for defaultExtensions improves readability without affecting logic. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
12. src/helpers.ts:4
- Draft comment:
Doc comment in mangleScopedPackage now spans multiple lines for better clarity. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
13. src/types.ts:5
- Draft comment:
Concise JSDoc formatting in the TypeScriptResolverOptions interface improves clarity; looks good. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
14. tests/e2e/base.eslintrc.cjs:4
- Draft comment:
Reformatted JSDoc @import comment in the ESLint config improves readability without changing functionality. - Reason this comment was not posted:
Confidence changes required:0%
<= threshold50%
None
15. README.md:51
- Draft comment:
Typographical issue: In the sentence 'After version 2.0.0,.d.ts
will take higher priority then normal.js
/.jsx
files...', 'then' should be replaced with 'than' to correctly indicate comparison. - 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_WRvJYfczZsNRng0U
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
I’m encountering an issue related to this pull request! I encountered a build error when using [email protected] in a project that utilizes Yarn Plug'n'Play (PnP). The error message indicates that napi-postinstall attempts to access tslib, but it is not declared as a direct dependency in the napi-postinstall package. This violates Yarn PnP's strict dependency resolution and prevents the build from succeeding. More info: #436 and unrs/unrs-resolver#71 |
| datasource | package | from | to | | ---------- | --------------------------------- | ----- | ----- | | npm | eslint-import-resolver-typescript | 4.3.1 | 4.3.3 | ## [v4.3.3](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#433) ##### Patch Changes - [#433](import-js/eslint-import-resolver-typescript#433) [`834b11e`](import-js/eslint-import-resolver-typescript@834b11e) Thanks [@JounQin](https://github.com/JounQin)! - chore: bump `unrs-resolver` to v1.6.0 ## [v4.3.2](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#432) ##### Patch Changes - [#427](import-js/eslint-import-resolver-typescript#427) [`dabba8e`](import-js/eslint-import-resolver-typescript@dabba8e) Thanks [@JounQin](https://github.com/JounQin)! - chore: bump `unrs-resolver` to v1.4.1
| datasource | package | from | to | | ---------- | --------------------------------- | ----- | ----- | | npm | eslint-import-resolver-typescript | 4.3.1 | 4.3.4 | ## [v4.3.4](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#434) ##### Patch Changes - [#442](import-js/eslint-import-resolver-typescript#442) [`57611d9`](import-js/eslint-import-resolver-typescript@57611d9) Thanks [@JounQin](https://github.com/JounQin)! - fix: add more extension aliases for ts source/declaration files - [#444](import-js/eslint-import-resolver-typescript#444) [`bd45fcd`](import-js/eslint-import-resolver-typescript@bd45fcd) Thanks [@JounQin](https://github.com/JounQin)! - fix(deps): bump `unrs-resolver` which resolves [#406](import-js/eslint-import-resolver-typescript#406), [#409](import-js/eslint-import-resolver-typescript#409), [#437](import-js/eslint-import-resolver-typescript#437) ## [v4.3.3](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#433) ##### Patch Changes - [#433](import-js/eslint-import-resolver-typescript#433) [`834b11e`](import-js/eslint-import-resolver-typescript@834b11e) Thanks [@JounQin](https://github.com/JounQin)! - chore: bump `unrs-resolver` to v1.6.0 ## [v4.3.2](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#432) ##### Patch Changes - [#427](import-js/eslint-import-resolver-typescript#427) [`dabba8e`](import-js/eslint-import-resolver-typescript@dabba8e) Thanks [@JounQin](https://github.com/JounQin)! - chore: bump `unrs-resolver` to v1.4.1
Important
Bump
unrs-resolver
to v1.6.0 and update related configurations and documentation.unrs-resolver
to v1.6.0 inpackage.json
.tinyglobby
to v0.2.13 and@1stg/common-config
to v13.0.1 inpackage.json
..lintstagedrc.js
to.nano-staged.js
and update import from@1stg/lint-staged/tsc
to@1stg/nano-staged/tsc
.eslint
to v9.25.0 andeslint-plugin-import-x
to v4.10.5 inpackage.json
.src/constants.ts
,src/helpers.ts
, andsrc/types.ts
.This description was created by
for 26b8583. You can customize this summary. It will automatically update as commits are pushed.
Summary by CodeRabbit