Skip to content

Add Roman Numeral in R - #6095

Merged
rzuckerm merged 2 commits into
TheRenegadeCoder:mainfrom
msj4:patch-3
Oct 20, 2026
Merged

rzuckerm merged 2 commits into
TheRenegadeCoder:mainfrom
msj4:patch-3

Conversation

@msj4

@msj4 msj4 commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

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.

Please fill in the checkbox like this:

[x]

Not like these:

[x ]
[ x]
[ x ]

Please put in the relevant issue number in the I fixed #your-issue-number-here item. For example, if your PR is
for an issue called Add Convex Hull in C++, and that issue number is 5164, change your-issue-number-here
to 5164. Do not put any space between # and the issue number. This is important because when the PR is merged,
the corresponding issue will be closed.

For more information please refer to our contributing documentation

I Am Adding a New Code Snippet in an Existing Language

  • I fixed Add Roman Numeral in R #6094
  • I did not include any extra folders/libraries
  • I named the pull request using Add {PROJECT} in {LANGUAGE} format

@github-actions github-actions Bot added enhancement Any code that improves the repo roman numeral See: https://sampleprograms.io/projects/roman-numeral/ labels Oct 18, 2026
@msj4

msj4 commented Oct 18, 2026

Copy link
Copy Markdown
Contributor Author

not able to see, which test case is failing. Please help.

@rzuckerm rzuckerm changed the title Create roman-numeral.r Add Roman Numeral in R Oct 18, 2026
@rzuckerm

Copy link
Copy Markdown
Collaborator

not able to see, which test case is failing. Please help.

The filename would end with .R, not .r. Also, I corrected the title and description of the PR. For future PRs, please fill this in properly.

@rzuckerm rzuckerm left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Wrong filename

@rzuckerm

Copy link
Copy Markdown
Collaborator

If you have not already, I encourage you to read our contributing document.

@rzuckerm rzuckerm left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I renamed the file. Thanks for your submission.

@rzuckerm
rzuckerm merged commit c6fbf2c into TheRenegadeCoder:main Oct 20, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Any code that improves the repo roman numeral See: https://sampleprograms.io/projects/roman-numeral/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add Roman Numeral in R

2 participants