Skip to content

fix: replace new Array() with array literal#10940

Open
riyakapoor-tech wants to merge 1 commit intostdlib-js:developfrom
riyakapoor-tech:patch-3
Open

fix: replace new Array() with array literal#10940
riyakapoor-tech wants to merge 1 commit intostdlib-js:developfrom
riyakapoor-tech:patch-3

Conversation

@riyakapoor-tech
Copy link
Copy Markdown

Resolves #10945

Description

Replaced the use of the new Array() constructor with an array literal
to comply with the stdlib/no-new-array lint rule in

lib/node_modules/@stdlib/process/umask/scripts/binary_to_symbolic.js.

  • Read, understood, and followed the contributing guidelines

Signed-off-by: riyakapoor-tech <riyanitc24@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 First-time Contributor A pull request from a contributor who has never previously committed to the project repository. Needs Review A pull request which needs code review. labels Mar 14, 2026
@Aniket-SS
Copy link
Copy Markdown
Contributor

The issue #10945 you referenced does not exist yet.

@stdlib-bot stdlib-bot added Potential Duplicate There might be another pull request resolving the same issue. and removed Potential Duplicate There might be another pull request resolving the same issue. labels Mar 16, 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. Needs Review A pull request which needs code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix C lint errors

3 participants