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.

Sign up for GitHub

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

Jump to bottom

[isogram] remove unicode testcase #297

Closed
NobbZ opened this issue Feb 20, 2017 · 1 comment
Closed

[isogram] remove unicode testcase #297

NobbZ opened this issue Feb 20, 2017 · 1 comment
Labels
good first issue 🐥

Comments

@NobbZ
Copy link
Member

NobbZ commented Feb 20, 2017

In exercism/problem-specifications#428 there was found consens in not testing behaviour for something outside ASCII for a couple of reasons.

The current testsuite of isograms violates this rule by the last test case.
@parkerl parkerl changed the title [isogram] unicode testcase [isogram] remove unicode testcase Feb 25, 2017
@stale
Copy link

stale bot commented Apr 29, 2017

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

awochna added a commit to awochna/xelixir that referenced this issue May 4, 2017
@awochna
Remove UTF-8 tests, as per exercism#297
b8b689f
@awochna awochna mentioned this issue May 4, 2017
Merged
parkerl added a commit that referenced this issue May 8, 2017
@parkerl
Merge pull request #334 from awochna/297-isogram-unicode
fb4709b 
Remove UTF-8 tests, as per #297
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue 🐥
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@NobbZ @parkerl @devonestes