9) All Longest Strings #4

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.

Sign up for GitHub

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

Merged

jakubjirous merged 1 commit into main from feature/all-longest-strings

Jan 7, 2023

Owner

jakubjirous commented Jan 7, 2023 •

edited

Loading

All Longest Strings

Problem Description @ Code Signal

PR Checklist:

Understand the problem
- You know what is known
- You know what is unknown
- You thought throw some edge cases that could potentially happen
Devise a plan
- Writing a comment to code challenge what you want to do
- Showing the ability to take that problem and break it down to the parts
- Take the larger problem and take it down into smaller problems
Execute the plan
- Turning into an actual code
Review your solution
- Time to consider if there were a better approach
- Backwards realizing room for improvements
Unit test
- Cover your result with appropriate assertions

@octocat 👍 This PR looks great - it's ready to merge!

github-actions bot added the code-challenge label

github-actions bot assigned jakubjirous

9) all longest strings

ee51af2

9) all longest strings

jakubjirous force-pushed the feature/all-longest-strings branch from e18dcf4 to ee51af2 Compare

January 7, 2023 20:12

jakubjirous enabled auto-merge

January 7, 2023 20:13

sonarqubecloud bot commented Jan 7, 2023

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

jakubjirous merged commit dd15928 into main

jakubjirous deleted the feature/all-longest-strings branch

January 7, 2023 20:14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment