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

Sync roman-numerals tests #356

Merged
merged 2 commits into from
Apr 24, 2024
Merged

Conversation

BNAndras
Copy link
Member

Related to #229.

I swapped out equal for string=, updated the test names, and added the latest test for 3999. I also reformatted the stub to resemble what the exercise generator would make nowadays. Finally, I looked up the original author to give them proper credit.

@BNAndras BNAndras changed the title Sync roman numerals Sync roman-numerals tests Apr 22, 2024
@BNAndras BNAndras requested a review from ErikSchierboom April 23, 2024 13:47
@BNAndras BNAndras added x:action/sync Sync content with its latest version x:rep/tiny Tiny amount of reputation labels Apr 23, 2024
@BNAndras BNAndras merged commit 2c79d07 into exercism:main Apr 24, 2024
3 checks passed
@BNAndras BNAndras deleted the sync-roman-numerals branch April 24, 2024 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
x:action/sync Sync content with its latest version x:rep/tiny Tiny amount of reputation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants