eslint-import-resolver-typescript icon indicating copy to clipboard operation
eslint-import-resolver-typescript copied to clipboard

Allow either import or import-x plugin as a peer dependency

Open RobinTail opened this issue 1 year ago • 5 comments

Fixes #293

The PR marks both import and import-x plugins as optional peer dependencies, thus disables the warning on missing import when you're using import-x instead.

RobinTail avatar Jun 25 '24 10:06 RobinTail

🦋 Changeset detected

Latest commit: c370a299629f88ea89e99646f25164f568b6689a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
eslint-import-resolver-typescript Patch

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

changeset-bot[bot] avatar Jun 25 '24 10:06 changeset-bot[bot]

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.

codesandbox-ci[bot] avatar Jun 25 '24 10:06 codesandbox-ci[bot]

The CI fails to install Node 16 🤷🏽‍♀️

RobinTail avatar Jun 25 '24 10:06 RobinTail

I believe enabling corepack should go after installing Node in CI This should be fixed by:

  • https://github.com/import-js/eslint-import-resolver-typescript/pull/290

RobinTail avatar Jun 26 '24 18:06 RobinTail

@JounQin please review

RobinTail avatar Jul 04 '24 19:07 RobinTail

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] Transitive: environment, eval, filesystem, unsafe +44 7.92 MB jounqin

🚮 Removed packages: npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected]

View full report↗︎

socket-security[bot] avatar Jul 14 '24 15:07 socket-security[bot]

Please release, @JounQin

RobinTail avatar Jul 14 '24 20:07 RobinTail

I found that release failed: https://github.com/import-js/eslint-import-resolver-typescript/actions/runs/9928981640/job/27425949199

RobinTail avatar Jul 14 '24 20:07 RobinTail

@JounQin Can you fix this? Thanks 😊

carlocorradini avatar Jul 17 '24 09:07 carlocorradini

error npm error 404 '[email protected]' is not in this registry.

Perhaps network issue and just need to be triggered again, @JounQin

RobinTail avatar Jul 17 '24 11:07 RobinTail

error npm error 404 '[email protected]' is not in this registry.

Perhaps network issue and just need to be triggered again, @JounQin

I've tried to trigger the re-run of the failed job, but also failed.

SukkaW avatar Jul 17 '24 14:07 SukkaW

Then I'd assume that the token is expired, @SukkaW || @JounQin

secrets.NPM_TOKEN

or try

npm publish

instead of yarn release

Made a PR: https://github.com/import-js/eslint-import-resolver-typescript/pull/304

RobinTail avatar Jul 17 '24 14:07 RobinTail