-
Notifications
You must be signed in to change notification settings - Fork 19.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Pangram.java #6048
base: master
Are you sure you want to change the base?
Update Pangram.java #6048
Conversation
Updated the Pangram.java file by adding a bitwise approach and improving code readability
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6048 +/- ##
============================================
- Coverage 72.42% 72.37% -0.06%
+ Complexity 4991 4990 -1
============================================
Files 655 655
Lines 17593 17602 +9
Branches 3388 3392 +4
============================================
- Hits 12742 12739 -3
- Misses 4371 4381 +10
- Partials 480 482 +2 ☔ View full report in Codecov by Sentry. |
Hello everyone, I am ne in this contributing world, just checking in on my pull request. I'm excited to learn from your feedback and improve my contributions. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please replace main with JUnit tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I replaced main with JUnit tests. Please check and review . Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the Pangram.java file by adding a bitwise approach and improving code readability
clang-format -i --style=file path/to/your/file.java