fix: replace new Array() constructor in utils/async/try-then#11185
fix: replace new Array() constructor in utils/async/try-then#11185Muhammad-Nur-Alamsyah-Anwar wants to merge 1 commit intostdlib-js:developfrom
Conversation
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
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 |
ccd8b13 to
42fad7e
Compare
Signed-off-by: Muhammad Nur Alamsyah Anwar <muhammadnuralamsyahanwar@gmail.com>
42fad7e to
aa7df7a
Compare
Resolves #11183.
Description
This pull request:
new Array()constructor with array literal approach in@stdlib/utils/async/try-thento fixstdlib/no-new-arraylint errors at lines 99 and 123 inmain.jsRelated Issues
This pull request has the following related issues:
Questions
No.
Other
No.
Checklist
AI Assistance
@stdlib-js/reviewers