-
-
Notifications
You must be signed in to change notification settings - Fork 424
Added FizzBuzz,WaterBottle,SignOf and BrokenCal Solution In Java #203
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
Conversation
I can tell this is your first pull request! Thank you I'm so honored. 🎉🎉🎉 I'll take a look at it ASAP! |
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.
- Don't add main methods
- Remove un-necessary package names
- Update README with Problem link and Time/Space Complexity
Okay @GouravRusiya30 I am sure do that |
I had made the changes would you please review it |
@Manish-kasera |
Updated Readme.md File.... Hope everything is Correct @GouravRusiya30 |
@GouravRusiya30 review it please. |
Pull Request Template
Description
Please include a summary of the problem and about the approach to solve the solution. Please also include relevant motivation and context.
List any dependencies that are required for this change.
Put check marks:
Have you made changes in README file ?
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.
Make sure all below guidelines are followed else PR will get Reject: