Skip to content

[RECREATED] Deal with different file encodings when submitting solutions#184

Merged
kytrinyx merged 3 commits intoexercism:masterfrom
ambroff:issue-2303
Apr 6, 2015
Merged

[RECREATED] Deal with different file encodings when submitting solutions#184
kytrinyx merged 3 commits intoexercism:masterfrom
ambroff:issue-2303

Conversation

@ambroff
Copy link
Copy Markdown
Contributor

@ambroff ambroff commented Apr 5, 2015

Recreating #182 because it is stuck closed after I recreated the branch.

Fixes exercism/exercism#2303

ambroff added 3 commits April 3, 2015 17:51
Fixes issue #2303 in the exercism.io project.
exercism/exercism#2303

Attempt to detect the charset of a submission so that it can be properly
transcoded to UTF-8.

Added UTF-16 and UTF-16BE fixture files for the test suite.

Changed the tests slightly to deal with different path and newline
separators on Windows.

Update travis config to include new dependencies.
Comment thread .travis.yml
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think go get -t ./... suffice here. It would fetch all dependencies for each package.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah, probably. I'll pull this in and then tweak that in a branch to make sure it works.

kytrinyx added a commit that referenced this pull request Apr 6, 2015
[RECREATED] Deal with different file encodings when submitting solutions
@kytrinyx kytrinyx merged commit d4dd8a3 into exercism:master Apr 6, 2015
@kytrinyx
Copy link
Copy Markdown
Member

kytrinyx commented Apr 6, 2015

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.

Submitting solutions that are UTF-16 encoded results in a 500 response

3 participants