Skip to content

Conversation

the-yash-rajput
Copy link
Contributor

@the-yash-rajput the-yash-rajput commented Oct 3, 2025

  • I have read CONTRIBUTING.md.
  • This pull request is all my own work -- I have not plagiarized it.
  • All filenames are in PascalCase.
  • All functions and variable names follow Java naming conventions.
  • All new algorithms have a URL in their comments that points to Wikipedia or other similar explanations.
  • All new code is formatted with clang-format -i --style=file path/to/your/file.java

@codecov-commenter
Copy link

codecov-commenter commented Oct 3, 2025

Codecov Report

❌ Patch coverage is 89.47368% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.30%. Comparing base (5f8d8ee) to head (115d87f).

Files with missing lines Patch % Lines
.../java/com/thealgorithms/maths/PiApproximation.java 89.47% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6602      +/-   ##
============================================
+ Coverage     76.29%   76.30%   +0.01%     
- Complexity     5914     5919       +5     
============================================
  Files           709      710       +1     
  Lines         20111    20130      +19     
  Branches       3899     3902       +3     
============================================
+ Hits          15343    15360      +17     
- Misses         4177     4179       +2     
  Partials        591      591              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@the-yash-rajput
Copy link
Contributor Author

@alxkm Can you please review this one too?

Copy link
Member

@DenizAltunkapan DenizAltunkapan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thank you!

@DenizAltunkapan DenizAltunkapan enabled auto-merge (squash) October 8, 2025 12:13
@DenizAltunkapan DenizAltunkapan merged commit b031a0b into TheAlgorithms:master Oct 8, 2025
6 checks passed
@the-yash-rajput the-yash-rajput deleted the PiApproximation branch October 9, 2025 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants