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 anagram with problem-specifications #1844

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Sync anagram with problem-specifications #1844

merged 1 commit into from
Jan 5, 2024

Conversation

senekor
Copy link
Contributor

@senekor senekor commented Dec 23, 2023

No description provided.

@senekor senekor requested a review from ErikSchierboom January 4, 2024 20:38
@senekor senekor merged commit f03997c into main Jan 5, 2024
11 checks passed
@senekor senekor deleted the anagram-sync branch January 5, 2024 14:55
senekor added a commit that referenced this pull request Jan 24, 2024
I removed some unicode related tests by accident in PR #1844. commit:
f03997c

When I noticed this, I upstreamed them to problem-specifications
to include them again here.
senekor added a commit that referenced this pull request Jan 24, 2024
I accidentally removed some unicode related tests in PR #1844. These are
now upstreamed to problem-specifications.
@ler0sQ
Copy link

ler0sQ commented Jan 24, 2024

@senekor According to the problem's description there'll be ONLY ASCII alphabetic characters. What am I missing? (I'm no developer here, just a bit frustrated problem solver, who found himself in the middle of multiple changes)

@senekor
Copy link
Contributor Author

senekor commented Jan 24, 2024

@ler0sQ you're right, there's a mismatch between the instructions and the unicode tests. I can't change the instructions, they're coming from upspream. But I can make track-specific additions at the end. Would it fix the confusion to add this sentence at the end of the instructions:

The Rust track extends the possible letters to be any unicode character, not just ASCII alphabetic ones.

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