Skip to content

Conversation

Payal-Goswami
Copy link
Contributor

Resolves #8183.

Description

This pull request fixes:

  • File: lib/node_modules/@stdlib/utils/nonenumerable-properties-in/lib/main.js

Added a missing require statement for Object to satisfy the stdlib/require-globals lint rule.

  • File: lib/node_modules/@stdlib/stats/ztest/examples/index.js

Replaced the prohibited new Array() constructor with an array literal and loop push.

Related Issues

This pull request:

Questions

No.

Other

No.

Checklist

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


@stdlib-js/reviewers

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

Hello! 👋

We've noticed that you've been opening a number of PRs addressing good first issues. Thank you for your interest and enthusiasm!

Now that you've made a few contributions, we suggest no longer working on good first issues. Instead, we encourage you to prioritize cleaning up any PRs which have yet to be merged and then proceed to work on more involved tasks.

Not only does this ensure that other new contributors can work on things and get ramped up on all things stdlib, it also ensures that you can spend your time on more challenging problems. 🚀

For ideas for future PRs, feel free to search the codebase for TODOs and FIXMEs and be sure to check out other open issues on the issue tracker. Cheers!

@kgryte kgryte changed the title Fix lint errors 8183 chore: fix lint errors (#8183) Oct 2, 2025
Signed-off-by: Athan <kgryte@gmail.com>
@stdlib-bot
Copy link
Contributor

stdlib-bot commented Oct 2, 2025

Coverage Report

Package Statements Branches Functions Lines
stats/ztest $\color{green}396/396$
$\color{green}+100.00\%$
$\color{green}62/62$
$\color{green}+100.00\%$
$\color{green}3/3$
$\color{green}+100.00\%$
$\color{green}396/396$
$\color{green}+100.00\%$
utils/nonenumerable-properties-in $\color{green}132/132$
$\color{green}+100.00\%$
$\color{green}13/13$
$\color{green}+100.00\%$
$\color{green}1/1$
$\color{green}+100.00\%$
$\color{green}132/132$
$\color{green}+100.00\%$

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

Signed-off-by: Athan <kgryte@gmail.com>
@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 Oct 2, 2025
Payal-Goswami and others added 6 commits October 3, 2025 11:46
Merge branch 'develop' of https://github.com/stdlib-js/stdlib into fix-lint-errors-8183
Merge branch 'fix-lint-errors-8183' of https://github.com/Payal-Goswami/stdlib into fix-lint-errors-8183
Signed-off-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.com>
kgryte added 2 commits October 3, 2025 11:18
Signed-off-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.com>
Signed-off-by: Athan <kgryte@gmail.com>
@kgryte kgryte removed the Needs Changes Pull request which needs changes before being merged. label Oct 3, 2025
@kgryte kgryte merged commit eebec21 into stdlib-js:develop Oct 3, 2025
13 checks passed
@Payal-Goswami
Copy link
Contributor Author

Thanks for merging! 🙌 I appreciate the adjustments you made - I’ll take note of them when working on future contributions.

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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix JavaScript lint errors
3 participants