Skip to content

Conversation

miladsade96
Copy link
Contributor

  • 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 Sep 29, 2025

Codecov Report

❌ Patch coverage is 96.96970% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 75.21%. Comparing base (f9a9ccb) to head (232ddf5).

Files with missing lines Patch % Lines
...va/com/thealgorithms/strings/KnuthMorrisPratt.java 96.96% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #6554      +/-   ##
============================================
+ Coverage     75.18%   75.21%   +0.03%     
- Complexity     5615     5624       +9     
============================================
  Files           690      691       +1     
  Lines         19408    19441      +33     
  Branches       3756     3764       +8     
============================================
+ Hits          14592    14623      +31     
  Misses         4248     4248              
- Partials        568      570       +2     

☔ 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.

@miladsade96 miladsade96 marked this pull request as ready for review September 30, 2025 08:56
@DenizAltunkapan
Copy link
Member

Thank you for your contribution! However, the KMP algorithm is already implemented in this repository under src/main/java/com/thealgorithms/strings/KMP.java. Please check the DIRECTORY.md file before opening a PR to ensure that an implementation does not already exist.

@miladsade96 miladsade96 deleted the dev_knuthMorrisPratt branch October 1, 2025 08:56
@miladsade96
Copy link
Contributor Author

@DenizAltunkapan
OK, Thanks

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