Skip to content
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

Added java solution for 14, 55 / python solution for 2420 #61

Merged
merged 13 commits into from Oct 2, 2022

Conversation

green-study
Copy link
Contributor

@green-study green-study commented Sep 30, 2022

Added number 14(Longest_Common_Prefix) for java
Added number 2420(Find All Good Indices) for python
Added number 55(Jump_Game) for java
If any formatting is to be done, kindly inform.

Thank you.

Copy link
Owner

@qiyuangong qiyuangong left a comment

Choose a reason for hiding this comment

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

LGTM

@qiyuangong
Copy link
Owner

Nice PR. :)

@qiyuangong qiyuangong merged commit 183f57f into qiyuangong:master Oct 2, 2022
@qiyuangong
Copy link
Owner

The only suggestion is that when starting a new pull request. Please check out a new branch from latest main/master (need to fetch and rebase with upstream). That will make PR and message clean.

@LONGNEW
Copy link
Contributor

LONGNEW commented Oct 2, 2022

The only suggestion is that when starting a new pull request. Please check out a new branch from latest main/master (need to fetch and rebase with upstream). That will make PR and message clean.

@qiyuangong Thank you for the advice.

@green-study green-study changed the title Added java solution for 14 / python solution for 2420 Added java solution for 14, 55 / python solution for 2420 Dec 4, 2022
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.

None yet

4 participants