chore: fix JavaScript lint errors (issue #11209)#11211
chore: fix JavaScript lint errors (issue #11209)#11211JE4NVRG wants to merge 1 commit intostdlib-js:developfrom
Conversation
- regexp_require.js: add empty line between require statement and code in JSDoc examples (stdlib/jsdoc-example-require-spacing) - async/until/main.js: replace new Array() with array literal and push (stdlib/no-new-array) - strided/ops/sub/scripts/index.js: remove unused eslint-disable directive for node/shebang - constants/float64/pinf/lib/index.js: remove unused eslint-disable directive for stdlib/require-globals Resolves stdlib-js#11209
|
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:
This acknowledgment confirms that you've read the guidelines, which include:
We can't review or accept contributions without this acknowledgment. Thank you for your understanding and cooperation. We look forward to reviewing your contribution! |
|
👋 Hi there! 👋 And thank you for opening your first pull request! We will review it shortly. 🏃 💨 Getting Started
Next Steps
Running Tests LocallyYou can use # Run tests for all packages in the math namespace:
make test TESTS_FILTER=".*/@stdlib/math/.*"
# Run benchmarks for a specific package:
make benchmark BENCHMARKS_FILTER=".*/@stdlib/math/base/special/sin/.*"If you haven't heard back from us within two weeks, please ping us by tagging the "reviewers" team in a comment on this PR. If you have any further questions while waiting for a response, please join our Zulip community to chat with project maintainers and other community members. We appreciate your contribution! Documentation Links |
Description
Fixes issue #11209 by resolving 4 JavaScript lint errors detected in the automated lint workflow.
Changes
repl/lib/regexp_require.jsutils/async/until/lib/main.jsnew Array()not allowed[]+.push()strided/ops/sub/scripts/index.jsconstants/float64/pinf/lib/index.jsTesting
All lint errors resolved. Run
npm run lintto verify.Related Issue
Fixes #11209
Let me know if you need any changes! 🚀