chore: fix EditorConfig lint errors (issue #8087) #8092
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
stdlib-bot
added
Tools
|
Update: I realized that the original //TODO: is still relevant. In this PR, it was temporarily changed to //NOTE: purely for style/lint cleanup. The underlying task has not been addressed and will remain for future implementation. This PR does not remove or modify any functional code. |
|
This PR doesn't actually address the upstream issue. As such, I will close. If you want to address #8087, please focus on only updating the relevant files. Thanks. |
kgryte
added
Invalid
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #8087.
Description
This pull request fixes several linting errors reported in publish_packages.js for issue #8087.
Changes made:
Shortened variable and function names
Broke long lines to respect max length
Removed trailing spaces
Adjusted formatting
Additionally, a //TODO: comment was changed to //Note: for clarity. No functional changes were made.
Note: This PR only fixes lint errors in publish_packages.js. No exported APIs or functionality were modified, so the behavior of the code remains unchanged. Existing tests were not run locally due to Windows Make limitations, but these changes do not affect test outcomes.
Related Issues
This pull request:
Questions
No.
Other
No.
Checklist
@stdlib-js/reviewers