Skip to content

Conversation

viet2307
Copy link
Contributor

Important

Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

@viet2307
Copy link
Contributor Author

@neetcode-gh
First of all, I want to express my gratitude for everything you have done and still doing for the community.

I am studying DSA and building my resume/github repo along the way for my end goal of getting hire by Google, so your neetcode.io, your youtube channel and now this github repo really, really helps a lot.
I am gradually doing 1-2 leetcode problem a day using Java, so I will slowly adding newer solutions for the remaining problems.

I am sorry to impose, but it would be much encouraging if we can connect on Linkedin.
It is kinda weird (and unprofessional too) to say all this in a PR comment section, but please forgive me for that.

Once again, thank you very much.

Copy link
Collaborator

@sujal-goswami sujal-goswami left a comment

Choose a reason for hiding this comment

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

Please Read Contributing.md carefully. your PR name does not follow the contributing guidelines.

Copy link
Contributor

@chetannada chetannada left a comment

Choose a reason for hiding this comment

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

make your commit name start with Create and your Leetcode Question name and end with .java for ex: Create 0080-remove-duplicates-from-sorted-array-II.java

@viet2307 viet2307 changed the title add Java solution for leetcode 80 Create: 0080-remove-duplicates-from-sorted-array-II.java Oct 11, 2023
@viet2307
Copy link
Contributor Author

Hi guys, so sorry for the very late reply. Somehow git did not sent me any notifications about this and it totally slipped my mind.
Anyway, I changed the PR name according to the guidelines. Please give it a look if you have the time.
Thank you very much!

@Ykhan799
Copy link
Collaborator

This solution is identical to the current Java solution in the repo, will close out this PR.

@Ykhan799 Ykhan799 closed this Oct 20, 2024
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.

4 participants