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

Updating tests for Anagram #1036

Merged
merged 4 commits into from
Jan 26, 2025
Merged

Conversation

jagdish-15
Copy link
Contributor

Pull Request: Syncing Tests for Anagram Exercise

This pull request updates the test suite for the Anagram exercise to sync it with the problem specification repository.

Copy link
Member

@ryanplusplus ryanplusplus left a comment

Choose a reason for hiding this comment

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

I think that we should omit these new tests for the reasons that @wolf99 gave in #1037.

@jagdish-15
Copy link
Contributor Author

jagdish-15 commented Jan 26, 2025

Hi @ryanplusplus,

I've implemented the same changes as those in #1037. I’d appreciate it if you could review the PR. However, it seems there's an issue with the check step after incorporating the suggestions from @wolf99. Could you please take a look and advise?

[Edit- The issue in #1037 has been fixed now]

@ryanplusplus ryanplusplus merged commit 721f648 into exercism:main Jan 26, 2025
4 checks passed
@ryanplusplus
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants