-
-
Notifications
You must be signed in to change notification settings - Fork 55
chore: release eslint-plugin-import-x #407
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: master
Are you sure you want to change the base?
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
|
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. |
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 a9e53ac in 1 minute and 47 seconds. Click for details.
- Reviewed
39lines of code in3files - Skipped
0files when reviewing. - Skipped posting
3draft 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. CHANGELOG.md:7
- Draft comment:
Changelog update looks good – patch details and contributor info are clear. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
2. package.json:16
- Draft comment:
Top-level 'types' field (lib/index.d.cts) differs from the ESM export's types ('./lib/index.d.ts'). Confirm this is intentional to avoid type definition mismatches. - 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.
3. package.json:67
- Draft comment:
Consider specifying a semver range for peerDependency 'eslint-import-resolver-node' instead of '*' to help ensure compatibility. - Reason this comment was not posted:
Confidence changes required:50%<= threshold50%None
Workflow ID: wflow_8WhMBkh6dIb75mUo
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
925f7c7 to
183b080
Compare
183b080 to
9ef8728
Compare
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 9ef8728 in 1 minute and 2 seconds. Click for details.
- Reviewed
52lines of code in4files - Skipped
0files when reviewing. - Skipped posting
2draft 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. CHANGELOG.md:7
- Draft comment:
The new 4.16.2 release notes look well formatted. Confirm that removal of older changeset details is intentional. - Reason this comment was not posted:
Confidence changes required:0%<= threshold50%None
2. package.json:3
- Draft comment:
Version bump to 4.16.2 is correct. Also, please verify that the dual type definition fields ('lib/index.d.cts' vs. 'lib/index.d.ts' in exports) are intentional. - Reason this comment was not posted:
Confidence changes required:33%<= threshold50%None
Workflow ID: wflow_gGGNNYwiARzwBJKp
You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.
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
eslint-plugin-import-x@4.16.2
Patch Changes
#434
a3aae61Thanks @stepankuzmin! - fix(deps): replace type-fest with @package-json/typesPackageJson types are imported in published declaration files (lib/rules/no-extraneous-dependencies.d.ts and lib/utils/read-pkg-up.d.ts), which causes TypeScript compilation errors for consumers who don't have skipLibCheck enabled. Replacing type-fest with the smaller @package-json/types package ensures the types are available to all consumers while reducing bundle size.
#406
d0a7816Thanks @marcalexiei! - fix(package): remove config and rules exports pointing to empty filesImportant
Update
eslint-plugin-import-xto v4.16.2, replacingtype-festwith@package-json/typesand removing empty file exports.type-festwith@package-json/typesinpackage.jsonto fix TypeScript errors and reduce bundle size.package.json.CHANGELOG.mdto reflect changes in version 4.16.2.This description was created by
for 9ef8728. You can customize this summary. It will automatically update as commits are pushed.