Skip to content

Conversation

alxkm
Copy link
Member

@alxkm alxkm commented Aug 29, 2024

Fixed few naming and typo issues.

  • 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 Aug 29, 2024

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 50.76%. Comparing base (d189c3a) to head (50b9134).

Files with missing lines Patch % Lines
...ava/com/thealgorithms/stacks/LargestRectangle.java 0.00% 2 Missing ⚠️
...s/datastructures/lists/MergeKSortedLinkedList.java 0.00% 1 Missing ⚠️
src/main/java/com/thealgorithms/strings/Upper.java 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5436      +/-   ##
============================================
+ Coverage     50.75%   50.76%   +0.01%     
  Complexity     3138     3138              
============================================
  Files           523      523              
  Lines         15120    15120              
  Branches       2880     2880              
============================================
+ Hits           7674     7676       +2     
+ Misses         7122     7121       -1     
+ Partials        324      323       -1     

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

@vil02 vil02 changed the title refactor: typo style: use proper spelling Aug 30, 2024
@alxkm alxkm merged commit 87cf891 into TheAlgorithms:master Aug 30, 2024
5 checks passed
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