fix: rename pow.h to pow_int.h to match package name#11232
fix: rename pow.h to pow_int.h to match package name#11232Md-Zaid45 wants to merge 2 commits intostdlib-js:developfrom
Conversation
Signed-off-by: zaid <mz1944680@gmail.com>
|
👋 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 |
| */ | ||
|
|
||
| #include "stdlib/math/base/special/fast/pow.h" | ||
| #include "stdlib/math/base/special/fast/pow_int.h" |
There was a problem hiding this comment.
This same update needs to happen in two more files that were missed:
examples/c/example.c(line 19): still includespow.hbenchmark/c/native/benchmark.c(line 19): still includespow.h
Both need to be updated to include pow_int.h; otherwise, they'll fail to compile.
There was a problem hiding this comment.
Thanks for catching that. I'll update both files
Coverage ReportNo coverage information available. |
2570741 to
397bc80
Compare
Resolves #11230.
Description
This pull request:
pow.htopow_int.hto match the package name (pow-int) in snake_case, as required by the naming convention.Related Issues
This pull request has the following related issues:
@stdlib/math/base/special/fast/pow-int#11230Questions
No.
Other
No.
Checklist
AI Assistance
@stdlib-js/reviewers