Skip to content

chore: bump unrs-resolver to v1.6.2 to fix false positive lint error #86

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

Merged
merged 1 commit into from
Apr 21, 2025

Conversation

JounQin
Copy link
Contributor

@JounQin JounQin commented Apr 21, 2025

🚀 Pull Request Template

Description

close import-js/eslint-import-resolver-typescript#437

Checklist

  • Commit messages should follow conventional commits as much as possible. After staging your changes please run npm run commit
  • Lint, TypeScript, Prettier and all tests passing - npm run lint && npm run tsc && npm run test-report
  • Extended the Storybook / README / documentation, if necessary

By the way, I recommend you to migrate eslint-plugin-import to modern and faster https://github.com/un-ts/eslint-plugin-import-x, I can raise a PR for you if you agree.

@mkosir mkosir self-requested a review April 21, 2025 19:17
@mkosir mkosir added bug Something isn't working devDependencies labels Apr 21, 2025
@mkosir mkosir merged commit 4c202b6 into mkosir:main Apr 21, 2025
@JounQin JounQin deleted the chore/bump_unrs-resolver branch April 21, 2025 19:21
@mkosir
Copy link
Owner

mkosir commented Apr 21, 2025

Hey @JounQin, huge thanks for the PR! 🚀

Copy link

🎉 This PR is included in version 1.7.292 🎉

The release is available on:

Your semantic-release bot 📦🚀

@JounQin
Copy link
Contributor Author

JounQin commented Apr 22, 2025

Hey @JounQin, huge thanks for the PR! 🚀

@mkosir

By the way, I recommend you to migrate eslint-plugin-import to modern and faster https://github.com/un-ts/eslint-plugin-import-x, I can raise a PR for you if you agree.

@mkosir
Copy link
Owner

mkosir commented Apr 22, 2025

I totally missed it, upgrading to that sounds awesome. Feel free to open a PR for it 🙌

@JounQin
Copy link
Contributor Author

JounQin commented Apr 22, 2025

I totally missed it, upgrading to that sounds awesome. Feel free to open a PR for it 🙌

Here we go: #87

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working devDependencies released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

no-unresolved error in v4.x
2 participants