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

Update reverse-string tests #139

Merged
merged 1 commit into from
Feb 3, 2024

Conversation

BNAndras
Copy link
Member

@BNAndras BNAndras commented Feb 3, 2024

The two new grapheme clusters tests weren't working so I skipped them to avoid bumping the difficulty.

@BNAndras BNAndras added x:action/sync Sync content with its latest version x:rep/tiny Tiny amount of reputation labels Feb 3, 2024
@BNAndras
Copy link
Member Author

BNAndras commented Feb 3, 2024

CI fails because of an issue with the no important files action, but we're adding tests so we do want to rerun solutions here.

@BNAndras BNAndras merged commit ee228e7 into exercism:main Feb 3, 2024
2 of 3 checks passed
@BNAndras BNAndras deleted the update-reverse-string branch February 3, 2024 19:53
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.

1 participant