Skip to content

Correct error message in IOUtils#35

Closed
io7m wants to merge 1 commit into
apache:masterfrom
io7m:master
Closed

Correct error message in IOUtils#35
io7m wants to merge 1 commit into
apache:masterfrom
io7m:master

Conversation

@io7m

@io7m io7m commented Apr 25, 2017

Copy link
Copy Markdown
Contributor

This corrects the wording of the error message in IOUtils: There's
no such word as "readed", and the author probably meant "expected"
as opposed to "excepted".

@PascalSchumacher

Copy link
Copy Markdown
Contributor

Thanks!

Please also update the tests, see https://travis-ci.org/apache/commons-io/jobs/225698576 for details.

@io7m

io7m commented Apr 25, 2017

Copy link
Copy Markdown
Contributor Author

Ah, yes, sorry, I mistakenly assumed that there weren't going to be tests based on the error message string. Fixing it now.

This corrects the wording of the error message in IOUtils: There's
no such word as "readed", and the author probably meant "expected"
as opposed to "excepted".
@coveralls

coveralls commented Apr 25, 2017

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 90.04% when pulling 84a26d1 on io7m:master into 156653e on apache:master.

@asfgit asfgit closed this in 3378280 Apr 25, 2017
@PascalSchumacher

Copy link
Copy Markdown
Contributor

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.

3 participants