Skip to content

Conversation

@apurvavats
Copy link
Contributor

@apurvavats apurvavats commented Oct 25, 2025

Congrats on taking the first step to contributing to the Sample Programs repository maintained by The Renegade Coder!
For simplicity, please make sure that your pull request includes one and only one contribution.

Please fill one of the sections below as applicable.
Please also add any other relevant information to the Notes section at the bottom.
You may delete or just ignore any other sections.
For more information please refer to our contributing documentation

I Am Adding a New Code Snippet in an Existing Language

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hey @apurvavats! Thanks for contributing to this project! We are a rather small team, so it may take some time to process this request. In the meantime, there are several ways you can make yourself a part of The Renegade Coder community. For instance, you can:

Thanks for your help!

Copy link
Collaborator

@rzuckerm rzuckerm left a comment

Choose a reason for hiding this comment

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

Please take a look at the project description for the appropriate usage message.

@apurvavats
Copy link
Contributor Author

I did the required changes, please review it

Copy link
Collaborator

@rzuckerm rzuckerm left a comment

Choose a reason for hiding this comment

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

What you had before was a lot closer than what you have now. All that needed to be done was to fix the usage message. Also, I'd recommend assigning the usage message to a variable so that you don't have to keep repeating that string.

@apurvavats
Copy link
Contributor Author

updated the message, please review again.

@apurvavats apurvavats requested a review from rzuckerm October 25, 2025 17:34
@apurvavats
Copy link
Contributor Author

After checking the same program in different languages what i concluded is the solution requires only the final output and not not the output after every iteration, so please confirm once what are the requirements.

@rzuckerm
Copy link
Collaborator

After checking the same program in different languages what i concluded is the solution requires only the final output and not not the output after every iteration, so please confirm once what are the requirements.

The output is just supposed to be the number of the person remaining.

Copy link
Collaborator

@rzuckerm rzuckerm left a comment

Choose a reason for hiding this comment

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

@apurvavats Thanks for your contribution! After looking at the Description section of this, I can see how that would be confusing. However, the Requirements and Testing section clarify what the expected output is. I'll take this up with the founder of this repo and see what he says.

@rzuckerm rzuckerm merged commit e60e1bc into TheRenegadeCoder:main Oct 25, 2025
7 checks passed
@apurvavats
Copy link
Contributor Author

apurvavats commented Oct 25, 2025

@rzuckerm can you please add the label hactoberfest accepted

@rzuckerm rzuckerm added enhancement Any code that improves the repo hacktoberfest Hacktoberfest related josephus problem See: https://sampleprograms.io/projects/josephus-problem/ labels Oct 25, 2025
@rzuckerm
Copy link
Collaborator

@rzuckerm can you please add the label hactoberfest accepted

The whole repo is marked with the "hacktoberfest" label, so it should count.

@apurvavats
Copy link
Contributor Author

Thank you so much

@rzuckerm
Copy link
Collaborator

@apurvavats The Josephus Problem description has been updated.

@apurvavats
Copy link
Contributor Author

@rzuckerm it still does not appear in my hactoberfest profile.

@rzuckerm
Copy link
Collaborator

@apurvavats Sorry, I don't know what to tell you. According to the Hacktobestfest rules:

[participating] Your PR/MRs must be in a repo tagged with the “hacktoberfest” topic, or have the “hacktoberfest-accepted” label.

Our repo is tagged with the hacktoberfest label, so this should apply. I submitted several PRs to this repo that got approved, and all mine eventually showed up in my profile.

@TheRenegadeCoder TheRenegadeCoder locked as resolved and limited conversation to collaborators Oct 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement Any code that improves the repo hacktoberfest Hacktoberfest related josephus problem See: https://sampleprograms.io/projects/josephus-problem/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Josephus Problem in C++

2 participants