/core

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

Add missing String->Char conversion (fix #73) #155

Merged
merged 1 commit into from Feb 2, 2015

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@jonathanhefner @evancz @sonnym
@jonathanhefner
Contributor

jonathanhefner commented Feb 2, 2015

A straight-forward fix for #73. I just added a missing call to Utils.chr.

Test coverage included.
@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz Feb 2, 2015

Member

Awesome, thanks for closing issues like this, it is very very appreciated!
Member

evancz commented Feb 2, 2015

Awesome, thanks for closing issues like this, it is very very appreciated!

evancz pushed a commit that referenced this pull request Feb 2, 2015

@process-bot
Merge pull request #155 from jonathanhefner/issue-73-fix-char-fromcode 
Add missing String->Char conversion (fix #73)
800d068

@evancz evancz merged commit 800d068 into elm:master Feb 2, 2015
1 check passed

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@sonnym

This comment has been minimized.

Show comment
Hide comment
@sonnym

sonnym Feb 2, 2015

Awesome! Thanks!

sonnym commented Feb 2, 2015

Awesome! Thanks!
@jonathanhefner

This comment has been minimized.

Show comment
Hide comment
@jonathanhefner

jonathanhefner Feb 2, 2015

Contributor

Happy to be of service. =)
Contributor

jonathanhefner commented Feb 2, 2015

Happy to be of service. =)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment