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

Adds tests for whitespace and lowercase characters for the "Run Length Encoding" exersize #624

Merged
merged 1 commit into from
Mar 7, 2017
Merged

Adds tests for whitespace and lowercase characters for the "Run Length Encoding" exersize #624

merged 1 commit into from
Mar 7, 2017

Conversation

DesHorsley
Copy link
Contributor

As per issue #613

@stevejb71
Copy link
Contributor

Could you say which exercise you are changing in the commit message.
My reading of the description is that whitespace is not encoded. Could you update it to be clearer, and also say what type of whitespace is handled (just ascii 32 I guess?)

@DesHorsley DesHorsley changed the title Adds tests for whitespace and lowercase characters Adds tests for whitespace and lowercase characters for the "Run Length Encoding" exersize Mar 5, 2017
@DesHorsley
Copy link
Contributor Author

Hi @stevejb71 ,
I've updated the commit message, @behrtam is planning to update the description to clarify whitespace usage as per #613

@robkeim robkeim mentioned this pull request Mar 6, 2017
Closed
4 tasks
@rbasso rbasso mentioned this pull request Mar 6, 2017
Closed
75 tasks
@kotp kotp merged commit a3cc0fa into exercism:master Mar 7, 2017
behrtam added a commit to behrtam/python that referenced this pull request Mar 8, 2017
@behrtam
run-length-encoding: Update test cases
c26277b 
This removes the unicode test cases ([x-common/428](exercism/problem-specifications#428), [x-common/434](exercism/problem-specifications#434)) and
adds the new white space and lowercase tests ([x-common/624](exercism/problem-specifications#624)).
behrtam added a commit to behrtam/python that referenced this pull request Mar 8, 2017
@behrtam
run-length-encoding: Update test cases
3f253b4 
This removes the unicode test cases (exercism/problem-specifications#428, exercism/problem-specifications#434) and
adds the new white space and lowercase tests (exercism/problem-specifications#624).
@behrtam behrtam mentioned this pull request Mar 8, 2017
Merged
behrtam added a commit to behrtam/python that referenced this pull request Mar 16, 2017
@behrtam
run-length-encoding: Update test cases
0076e5f 
This removes the unicode test cases (exercism/problem-specifications#428, exercism/problem-specifications#434) and
adds the new white space and lowercase tests (exercism/problem-specifications#624).
behrtam added a commit to exercism/python that referenced this pull request Mar 16, 2017
@behrtam
run-length-encoding: Update test cases (#425)
64fc738 
This removes the unicode test cases (exercism/problem-specifications#428, exercism/problem-specifications#434) and
adds the new white space and lowercase tests (exercism/problem-specifications#624).
@petertseng petertseng mentioned this pull request May 8, 2017
Merged
emcoding pushed a commit that referenced this pull request Nov 19, 2018
@hilary
Merge pull request #625 from hilary/slugs_everywhere
ad16d96 
generator: slugs everywhere [closes #624]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@DesHorsley @stevejb71 @kotp