Skip to content

fix: don't resolve not implemented node modules in bun #402

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 4 commits into from
Mar 24, 2025

Conversation

SunsetTechuila
Copy link
Contributor

No description provided.

Copy link

changeset-bot bot commented Mar 23, 2025

🦋 Changeset detected

Latest commit: 300f9b6

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

Copy link

socket-security bot commented Mar 23, 2025

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

Package New capabilities Transitives Size Publisher
npm/[email protected] None 0 82.5 kB feross
npm/[email protected] None 0 21.9 kB sunset_techuila
npm/[email protected] None 0 2.82 kB sindresorhus
npm/[email protected] None 0 56.6 kB ai
npm/[email protected] None 0 4.55 kB sindresorhus
npm/[email protected] None 0 27.8 kB benjamn
npm/[email protected] None 0 2.56 kB kevva
npm/[email protected] None 0 0 B
npm/[email protected] None 0 0 B
npm/[email protected] None 0 5.05 kB google-wombot
npm/[email protected] None 0 26.3 kB google-wombot
npm/[email protected] None 0 0 B
npm/[email protected] None 0 11.8 kB sindresorhus

View full report↗︎

@SunsetTechuila SunsetTechuila force-pushed the bun branch 2 times, most recently from a30f2ef to 565c354 Compare March 23, 2025 23:20
Copy link

codesandbox-ci bot commented Mar 23, 2025

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.

Copy link

pkg-pr-new bot commented Mar 23, 2025

Open in Stackblitz

npm i https://pkg.pr.new/import-js/eslint-import-resolver-typescript@402

commit: 300f9b6

@SunsetTechuila SunsetTechuila changed the title fix: don't resolve not implemented bun modules fix: don't resolve not implemented node modules in bun Mar 23, 2025
@JounQin JounQin merged commit f21bf15 into import-js:master Mar 24, 2025
20 checks passed
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Mar 28, 2025
| datasource | package                           | from  | to    |
| ---------- | --------------------------------- | ----- | ----- |
| npm        | eslint-import-resolver-typescript | 4.2.2 | 4.2.5 |


## [v4.2.5](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#425)

##### Patch Changes

-   [#410](import-js/eslint-import-resolver-typescript#410) [`ec59d22`](import-js/eslint-import-resolver-typescript@ec59d22) Thanks [@JounQin](https://github.com/JounQin)! - fix: absolute path aliasing should not be skipped


## [v4.2.4](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#424)

##### Patch Changes

-   [#407](import-js/eslint-import-resolver-typescript#407) [`6b183ff`](import-js/eslint-import-resolver-typescript@6b183ff) Thanks [@JounQin](https://github.com/JounQin)! - chore: migrate to rebranding `unrs-resolver` with new targets supported:

    -   `i686-pc-windows-msvc`
    -   `armv7-unknown-linux-musleabihf`
    -   `powerpc64le-unknown-linux-gnu`
    -   `s390x-unknown-linux-gnu`


## [v4.2.3](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#423)

##### Patch Changes

-   [#402](import-js/eslint-import-resolver-typescript#402) [`f21bf15`](import-js/eslint-import-resolver-typescript@f21bf15) Thanks [@SunsetTechuila](https://github.com/SunsetTechuila)! - fix: don't resolve not implemented node modules in `bun`

    `is-bun-module` is marked as `dependency`, again, for correctness, see [`isBunImplementedNodeModule`](https://github.com/SunsetTechuila/is-bun-module#isbunimplementednodemodulemodulename-bunversion) for more details

    For `Bun` users: you don't need to install `is-bun-module` any more but `bun: true` option is still required if you're running without `bun --bun` nor [`run#bun`](https://bun.sh/docs/runtime/bunfig#run-bun-auto-alias-node-to-bun) enabled
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Mar 28, 2025
| datasource | package                           | from  | to    |
| ---------- | --------------------------------- | ----- | ----- |
| npm        | eslint-import-resolver-typescript | 4.2.2 | 4.2.5 |


## [v4.2.5](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#425)

##### Patch Changes

-   [#410](import-js/eslint-import-resolver-typescript#410) [`ec59d22`](import-js/eslint-import-resolver-typescript@ec59d22) Thanks [@JounQin](https://github.com/JounQin)! - fix: absolute path aliasing should not be skipped


## [v4.2.4](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#424)

##### Patch Changes

-   [#407](import-js/eslint-import-resolver-typescript#407) [`6b183ff`](import-js/eslint-import-resolver-typescript@6b183ff) Thanks [@JounQin](https://github.com/JounQin)! - chore: migrate to rebranding `unrs-resolver` with new targets supported:

    -   `i686-pc-windows-msvc`
    -   `armv7-unknown-linux-musleabihf`
    -   `powerpc64le-unknown-linux-gnu`
    -   `s390x-unknown-linux-gnu`


## [v4.2.3](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#423)

##### Patch Changes

-   [#402](import-js/eslint-import-resolver-typescript#402) [`f21bf15`](import-js/eslint-import-resolver-typescript@f21bf15) Thanks [@SunsetTechuila](https://github.com/SunsetTechuila)! - fix: don't resolve not implemented node modules in `bun`

    `is-bun-module` is marked as `dependency`, again, for correctness, see [`isBunImplementedNodeModule`](https://github.com/SunsetTechuila/is-bun-module#isbunimplementednodemodulemodulename-bunversion) for more details

    For `Bun` users: you don't need to install `is-bun-module` any more but `bun: true` option is still required if you're running without `bun --bun` nor [`run#bun`](https://bun.sh/docs/runtime/bunfig#run-bun-auto-alias-node-to-bun) enabled
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Mar 29, 2025
| datasource | package                           | from  | to    |
| ---------- | --------------------------------- | ----- | ----- |
| npm        | eslint-import-resolver-typescript | 4.2.2 | 4.3.1 |


## [v4.3.1](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#431)

##### Patch Changes

-   [#425](import-js/eslint-import-resolver-typescript#425) [`2ced0ba`](import-js/eslint-import-resolver-typescript@2ced0ba) Thanks [@JounQin](https://github.com/JounQin)! - chore: bump `unrs-resolver` to v1.3.3


## [v4.3.0](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#430)

##### Minor Changes

-   [#423](import-js/eslint-import-resolver-typescript#423) [`2fcb947`](import-js/eslint-import-resolver-typescript@2fcb947) Thanks [@JounQin](https://github.com/JounQin)! - feat: throw error on malformed `tsconfig` reference


## [v4.2.7](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#427)

##### Patch Changes

-   [`aeb558f`](import-js/eslint-import-resolver-typescript@aeb558f) Thanks [@JounQin](https://github.com/JounQin)! - fix: add missing `index.d.cts` file


## [v4.2.6](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#426)

##### Patch Changes

-   [#417](import-js/eslint-import-resolver-typescript#417) [`c3f678b`](import-js/eslint-import-resolver-typescript@c3f678b) Thanks [@JounQin](https://github.com/JounQin)! - fix: `require` entry types, add `module-sync` entry


## [v4.2.5](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#425)

##### Patch Changes

-   [#410](import-js/eslint-import-resolver-typescript#410) [`ec59d22`](import-js/eslint-import-resolver-typescript@ec59d22) Thanks [@JounQin](https://github.com/JounQin)! - fix: absolute path aliasing should not be skipped


## [v4.2.4](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#424)

##### Patch Changes

-   [#407](import-js/eslint-import-resolver-typescript#407) [`6b183ff`](import-js/eslint-import-resolver-typescript@6b183ff) Thanks [@JounQin](https://github.com/JounQin)! - chore: migrate to rebranding `unrs-resolver` with new targets supported:

    -   `i686-pc-windows-msvc`
    -   `armv7-unknown-linux-musleabihf`
    -   `powerpc64le-unknown-linux-gnu`
    -   `s390x-unknown-linux-gnu`


## [v4.2.3](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#423)

##### Patch Changes

-   [#402](import-js/eslint-import-resolver-typescript#402) [`f21bf15`](import-js/eslint-import-resolver-typescript@f21bf15) Thanks [@SunsetTechuila](https://github.com/SunsetTechuila)! - fix: don't resolve not implemented node modules in `bun`

    `is-bun-module` is marked as `dependency`, again, for correctness, see [`isBunImplementedNodeModule`](https://github.com/SunsetTechuila/is-bun-module#isbunimplementednodemodulemodulename-bunversion) for more details

    For `Bun` users: you don't need to install `is-bun-module` any more but `bun: true` option is still required if you're running without `bun --bun` nor [`run#bun`](https://bun.sh/docs/runtime/bunfig#run-bun-auto-alias-node-to-bun) enabled
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Mar 29, 2025
| datasource | package                           | from  | to    |
| ---------- | --------------------------------- | ----- | ----- |
| npm        | eslint-import-resolver-typescript | 4.2.2 | 4.3.1 |


## [v4.3.1](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#431)

##### Patch Changes

-   [#425](import-js/eslint-import-resolver-typescript#425) [`2ced0ba`](import-js/eslint-import-resolver-typescript@2ced0ba) Thanks [@JounQin](https://github.com/JounQin)! - chore: bump `unrs-resolver` to v1.3.3


## [v4.3.0](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#430)

##### Minor Changes

-   [#423](import-js/eslint-import-resolver-typescript#423) [`2fcb947`](import-js/eslint-import-resolver-typescript@2fcb947) Thanks [@JounQin](https://github.com/JounQin)! - feat: throw error on malformed `tsconfig` reference


## [v4.2.7](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#427)

##### Patch Changes

-   [`aeb558f`](import-js/eslint-import-resolver-typescript@aeb558f) Thanks [@JounQin](https://github.com/JounQin)! - fix: add missing `index.d.cts` file


## [v4.2.6](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#426)

##### Patch Changes

-   [#417](import-js/eslint-import-resolver-typescript#417) [`c3f678b`](import-js/eslint-import-resolver-typescript@c3f678b) Thanks [@JounQin](https://github.com/JounQin)! - fix: `require` entry types, add `module-sync` entry


## [v4.2.5](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#425)

##### Patch Changes

-   [#410](import-js/eslint-import-resolver-typescript#410) [`ec59d22`](import-js/eslint-import-resolver-typescript@ec59d22) Thanks [@JounQin](https://github.com/JounQin)! - fix: absolute path aliasing should not be skipped


## [v4.2.4](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#424)

##### Patch Changes

-   [#407](import-js/eslint-import-resolver-typescript#407) [`6b183ff`](import-js/eslint-import-resolver-typescript@6b183ff) Thanks [@JounQin](https://github.com/JounQin)! - chore: migrate to rebranding `unrs-resolver` with new targets supported:

    -   `i686-pc-windows-msvc`
    -   `armv7-unknown-linux-musleabihf`
    -   `powerpc64le-unknown-linux-gnu`
    -   `s390x-unknown-linux-gnu`


## [v4.2.3](https://github.com/import-js/eslint-import-resolver-typescript/blob/HEAD/CHANGELOG.md#423)

##### Patch Changes

-   [#402](import-js/eslint-import-resolver-typescript#402) [`f21bf15`](import-js/eslint-import-resolver-typescript@f21bf15) Thanks [@SunsetTechuila](https://github.com/SunsetTechuila)! - fix: don't resolve not implemented node modules in `bun`

    `is-bun-module` is marked as `dependency`, again, for correctness, see [`isBunImplementedNodeModule`](https://github.com/SunsetTechuila/is-bun-module#isbunimplementednodemodulemodulename-bunversion) for more details

    For `Bun` users: you don't need to install `is-bun-module` any more but `bun: true` option is still required if you're running without `bun --bun` nor [`run#bun`](https://bun.sh/docs/runtime/bunfig#run-bun-auto-alias-node-to-bun) enabled
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants