-
Notifications
You must be signed in to change notification settings - Fork 6
Added solution for problem 1672 #29
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
base: main
Are you sure you want to change the base?
Conversation
Added the java solution for problem 1672 for Hactoberfest.
|
Hi, I have submitted the solution, but I don't see any action being taken till now, is this repo active? |
SanthoshS20
left a comment
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.
Hi @Ayan1089 , I'm okay with the solution. But I need the problem description and the problem description in the file. Please add it. I have attached a sample for it.
SanthoshS20
left a comment
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.
Please apply the change. I will merge it
Hi @Ayan1089. Currently, there is no Github Action is going on. I will create a issue for it. If you are interested please take that too. |
Added javadoc for the problem link and description for hactoberfest
Ayan1089
left a comment
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.
Refactored the solution with javadocs
I have done the changes can you merge it now? |
Yeah sure, you can assign the issue to me, meanwhile, can you merge the existing PR? |
|
@SanthoshS20 are you planning to merge it to main? |
Added the java solution for problem 1672 for Hactoberfest.