Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: report node when loc not found #700

Merged
merged 4 commits into from
Jan 15, 2025
Merged

Conversation

ntnyq
Copy link
Contributor

@ntnyq ntnyq commented Jan 13, 2025

This PR fixes #687.

Copy link

changeset-bot bot commented Jan 13, 2025

🦋 Changeset detected

Latest commit: 62005b2

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

This PR includes changesets to release 1 package
Name Type
eslint-plugin-prettier 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

@JounQin
Copy link
Member

JounQin commented Jan 13, 2025

@ntnyq I can't edit the PR directly because of missing permission, so please help to add a changeset, or enable Allow edits from maintainers.

@JounQin JounQin enabled auto-merge (squash) January 13, 2025 04:27
@JounQin JounQin disabled auto-merge January 13, 2025 04:28
@JounQin
Copy link
Member

JounQin commented Jan 13, 2025

Thanks for your contribution first! Can you add a test case for it?

@ntnyq
Copy link
Contributor Author

ntnyq commented Jan 13, 2025

Thanks for your contribution first! Can you add a test case for it?

Yes, I'll add the test case after I finish my lunch.

@ntnyq
Copy link
Contributor Author

ntnyq commented Jan 13, 2025

Sorry I failed to add a test case for this.

The original issue has mixed usage of esm and cjs syntax in prettier.config.cjs that causes the error.

I can't reproduce it in test even if I tried to add an integrated test.

BTW: Allow edits by maintainers feature is enabled now.

@JounQin JounQin merged commit aa5b59f into prettier:master Jan 15, 2025
4 checks passed
@JounQin
Copy link
Member

JounQin commented Jan 15, 2025

@ntnyq See d6b90c1 (#700) for adding a failing test case commit.

Thanks for your work!

renovate bot added a commit to mmkal/eslint-plugin-mmkal that referenced this pull request Jan 15, 2025
renovate bot added a commit to mmkal/eslint-plugin-mmkal that referenced this pull request Jan 15, 2025
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 17, 2025
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.2.1 | 5.2.2 |


## [v5.2.2](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#522)

##### Patch Changes

-   [#700](prettier/eslint-plugin-prettier#700) [`aa5b59f`](prettier/eslint-plugin-prettier@aa5b59f) Thanks [@ntnyq](https://github.com/ntnyq)! - fix: report node when loc not found
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 20, 2025
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.2.1 | 5.2.3 |


## [v5.2.3](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#523)

##### Patch Changes

-   [#703](prettier/eslint-plugin-prettier#703) [`9c6141f`](prettier/eslint-plugin-prettier@9c6141f) Thanks [@BPScott](https://github.com/BPScott)! - Add name field to recommended flat config


## [v5.2.2](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#522)

##### Patch Changes

-   [#700](prettier/eslint-plugin-prettier#700) [`aa5b59f`](prettier/eslint-plugin-prettier@aa5b59f) Thanks [@ntnyq](https://github.com/ntnyq)! - fix: report node when loc not found
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 24, 2025
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.2.1 | 5.2.3 |


## [v5.2.3](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#523)

##### Patch Changes

-   [#703](prettier/eslint-plugin-prettier#703) [`9c6141f`](prettier/eslint-plugin-prettier@9c6141f) Thanks [@BPScott](https://github.com/BPScott)! - Add name field to recommended flat config


## [v5.2.2](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#522)

##### Patch Changes

-   [#700](prettier/eslint-plugin-prettier#700) [`aa5b59f`](prettier/eslint-plugin-prettier@aa5b59f) Thanks [@ntnyq](https://github.com/ntnyq)! - fix: report node when loc not found
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 25, 2025
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.2.1 | 5.2.3 |


## [v5.2.3](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#523)

##### Patch Changes

-   [#703](prettier/eslint-plugin-prettier#703) [`9c6141f`](prettier/eslint-plugin-prettier@9c6141f) Thanks [@BPScott](https://github.com/BPScott)! - Add name field to recommended flat config


## [v5.2.2](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#522)

##### Patch Changes

-   [#700](prettier/eslint-plugin-prettier#700) [`aa5b59f`](prettier/eslint-plugin-prettier@aa5b59f) Thanks [@ntnyq](https://github.com/ntnyq)! - fix: report node when loc not found
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 26, 2025
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.2.1 | 5.2.3 |


## [v5.2.3](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#523)

##### Patch Changes

-   [#703](prettier/eslint-plugin-prettier#703) [`9c6141f`](prettier/eslint-plugin-prettier@9c6141f) Thanks [@BPScott](https://github.com/BPScott)! - Add name field to recommended flat config


## [v5.2.2](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#522)

##### Patch Changes

-   [#700](prettier/eslint-plugin-prettier#700) [`aa5b59f`](prettier/eslint-plugin-prettier@aa5b59f) Thanks [@ntnyq](https://github.com/ntnyq)! - fix: report node when loc not found
renovate bot added a commit to andrei-picus-tink/auto-renovate that referenced this pull request Jan 27, 2025
| datasource | package                | from  | to    |
| ---------- | ---------------------- | ----- | ----- |
| npm        | eslint-plugin-prettier | 5.2.1 | 5.2.3 |


## [v5.2.3](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#523)

##### Patch Changes

-   [#703](prettier/eslint-plugin-prettier#703) [`9c6141f`](prettier/eslint-plugin-prettier@9c6141f) Thanks [@BPScott](https://github.com/BPScott)! - Add name field to recommended flat config


## [v5.2.2](https://github.com/prettier/eslint-plugin-prettier/blob/HEAD/CHANGELOG.md#522)

##### Patch Changes

-   [#700](prettier/eslint-plugin-prettier#700) [`aa5b59f`](prettier/eslint-plugin-prettier@aa5b59f) Thanks [@ntnyq](https://github.com/ntnyq)! - fix: report node when loc not found
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Node must be provided when reporting error if location is not provided
2 participants