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

Solve many UVA problems #40

Merged
merged 160 commits into from
May 31, 2017
Merged

Solve many UVA problems #40

merged 160 commits into from
May 31, 2017

Conversation

fatosmorina
Copy link
Contributor

@fatosmorina fatosmorina commented May 11, 2017

This is a list of solutions to many UVA problems.

@kdn251
Copy link
Owner

kdn251 commented May 15, 2017

Hi @fatosmorina, thank you for the PR. Sorry for the late response, I am reviewing your solutions now!

@kdn251
Copy link
Owner

kdn251 commented May 26, 2017

Hi @fatosmorina I have begun reviewing your changes; however, before I continue I would like you to please change "static void solve()" to "public static void main(String args[])" for all of your files in order to maintain the consistency of the repository. Please let me know when you have done so and I will continue reviewing your PR. Thank you!

@fatosmorina
Copy link
Contributor Author

Hi @kdn251

I have made the changes you asked for. Can you please continue with the review now?

Thanks.

@kdn251
Copy link
Owner

kdn251 commented May 31, 2017

Hi @fatosmorina thank you so for your patience. I'm committing your changes now! Looking forward to more of your contributions :)

@kdn251 kdn251 merged commit 3016032 into kdn251:master May 31, 2017
@fatosmorina
Copy link
Contributor Author

Hi @kdn251

Thank you too for giving me the opportunity to contribute in your helpful repository. I am honored and privileged.

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.

2 participants