Skip to content

Fix EditorConfig lint errors #15088 - #15134

Open
Rajdeep0511 wants to merge 4 commits into
stdlib-js:developfrom
Rajdeep0511:develop
Open

Fix EditorConfig lint errors #15088#15134
Rajdeep0511 wants to merge 4 commits into
stdlib-js:developfrom
Rajdeep0511:develop

Conversation

@Rajdeep0511

@Rajdeep0511 Rajdeep0511 commented Sep 9, 2026

Copy link
Copy Markdown

Resolves #15088.
This pull request:

re-indents lines 39–51 of lib/node_modules/@stdlib/_tools/eslint/rules/no-builtin-math/examples/index.js with tabs instead of spaces.
The /* => ... / example-output block used 4-space indentation, while .editorconfig sets indent_style = tab for [.{js,js.txt}]. Each level is replaced by one tab at the same depth (4→1, 8→2, 12→3), so the rendered block is unchanged in structure.

Verified with editorconfig-checker against the repository's own .editorconfig:

BEFORE before.js:
39-51: wrong indentation type (spaces instead of tabs)
1 errors found

AFTER (no output, exit 0)
That matches the workflow error in the issue exactly. I also confirmed the change is whitespace-only: diff <(tr -d ' \t' < before.js) <(tr -d ' \t' < after.js) is empty.

Related Issues
This pull request has the following related issues:

#15088
Questions
No.

Other
No.

Checklist
Read, understood, and followed the contributing guidelines.
AI Assistance
When authoring the changes proposed in this PR, did you use any kind of AI assistance?

Yes
No
If you answered "yes" above, how did you use AI assistance?

Code generation (e.g., when writing an implementation or fixing a bug)
Test/benchmark generation
Documentation (including examples)
Research and understanding
I used an AI assistant to locate the flagged lines and apply the re-indentation, then verified the result myself by running editorconfig-checker against the repository's .editorconfig on both the original and the patched file, and by confirming the diff is whitespace-only.

@Rajdeep0511
Rajdeep0511 requested a review from a team September 9, 2026 19:35
@stdlib-bot stdlib-bot added Needs Review A pull request which needs code review. First-time Contributor A pull request from a contributor who has never previously committed to the project repository. labels Sep 9, 2026
@stdlib-bot

Copy link
Copy Markdown
Contributor

Hello! Thank you for your contribution to stdlib.

We noticed that the contributing guidelines acknowledgment is missing from your pull request. Here's what you need to do:

  1. Please read our contributing guidelines.

  2. Update your pull request description to include this checked box:

    - [x] Read, understood, and followed the [contributing guidelines](https://github.com/stdlib-js/stdlib/blob/develop/CONTRIBUTING.md)

This acknowledgment confirms that you've read the guidelines, which include:

  • The developer's certificate of origin
  • Your agreement to license your contributions under the project's terms

We can't review or accept contributions without this acknowledgment.

Thank you for your understanding and cooperation. We look forward to reviewing your contribution!

Signed-off-by: Rajdeep <palr73414@gmail.com>
@stdlib-bot

Copy link
Copy Markdown
Contributor

Hello! Thank you for your contribution to stdlib.

We noticed that the contributing guidelines acknowledgment is missing from your pull request. Here's what you need to do:

  1. Please read our contributing guidelines.

  2. Update your pull request description to include this checked box:

    - [x] Read, understood, and followed the [contributing guidelines](https://github.com/stdlib-js/stdlib/blob/develop/CONTRIBUTING.md)

This acknowledgment confirms that you've read the guidelines, which include:

  • The developer's certificate of origin
  • Your agreement to license your contributions under the project's terms

We can't review or accept contributions without this acknowledgment.

Thank you for your understanding and cooperation. We look forward to reviewing your contribution!

@stdlib-bot stdlib-bot added Good First PR A pull request resolving a Good First Issue. Potential Duplicate There might be another pull request resolving the same issue. labels Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

First-time Contributor A pull request from a contributor who has never previously committed to the project repository. Good First PR A pull request resolving a Good First Issue. Needs Review A pull request which needs code review. Potential Duplicate There might be another pull request resolving the same issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix EditorConfig lint errors

2 participants