Skip to content

Conversation

Shivam-1827
Copy link
Contributor

Resolves #6031

Description

What is the purpose of this pull request?

This pull request:

  • Fixes EditorConfig linting errors by replacing tabs with spaces in affected files.
  • Ensures adherence to the project's .editorconfig rules for indentation.
  • Updates the manifest.json file in lib/node_modules/@stdlib/number/float32/base/exponent/ to use spaces instead of tabs.

Related Issues

Does this pull request have any related issues?

This pull request:

Questions

Any questions for reviewers of this pull request?

No.

Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

  • Ran make lint-editorconfig-files locally to verify the fix.
  • Ensured no other formatting issues remain in the affected files.

Checklist

Please ensure the following tasks are completed before submitting this pull request.


@stdlib-js/reviewers

@stdlib-bot stdlib-bot added Needs Review A pull request which needs code review. Good First PR A pull request resolving a Good First Issue. labels Mar 17, 2025
@stdlib-bot
Copy link
Contributor

stdlib-bot commented Mar 17, 2025

Coverage Report

Package Statements Branches Functions Lines
number/float32/base/exponent $\color{green}199/199$
$\color{green}+100.00\%$
$\color{green}5/5$
$\color{green}+100.00\%$
$\color{green}2/2$
$\color{green}+100.00\%$
$\color{green}199/199$
$\color{green}+100.00\%$

The above coverage report was generated for the changes in this PR.

"resolve": true,
"relative": false
}
{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You have mixed tabs and spaces. This file should be 2-space indentation.

}
]
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are missing a trailing newline.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for pointing out my mistakes. I'll fix those errors

@kgryte kgryte added Needs Changes Pull request which needs changes before being merged. and removed Needs Review A pull request which needs code review. labels Mar 17, 2025
@Shivam-1827 Shivam-1827 deleted the fix/lint-error-6031 branch March 17, 2025 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Good First PR A pull request resolving a Good First Issue. Needs Changes Pull request which needs changes before being merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix EditorConfig lint errors

3 participants