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

Replace EOLTests with Test::EOL #33

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
3 participants
@paultcochrane
Contributor

paultcochrane commented Aug 27, 2016

The EOLTests Dist::Zilla plugin is deprecated and may be removed in the future. The recommended alternative is Test::EOL which is what has been used here. The files with trailing whitespace have been corrected and the end-of-line character tests now pass correctly.

This pull request is submitted in the hope that it is helpful. If you have any questions or comments regarding it, please don't hesitate to let me know.

paultcochrane added some commits Aug 27, 2016

Replace EOLTests with Test::EOL
The EOLTests Dist::Zilla plugin is deprecated and may be removed in the
future.  The recommended alternative is Test::EOL which is what has been
used here.
Purge trailing whitespace
Now the author-eol.t tests pass correctly.
@perlpunk

This comment has been minimized.

Show comment
Hide comment
@perlpunk

perlpunk Aug 27, 2016

Collaborator

@ingydotnet also a trivial change, merge it?

Collaborator

perlpunk commented Aug 27, 2016

@ingydotnet also a trivial change, merge it?

@ingydotnet

This comment has been minimized.

Show comment
Hide comment
@ingydotnet

ingydotnet Aug 29, 2016

Owner

Applied. Pushed. Thanks!

Owner

ingydotnet commented Aug 29, 2016

Applied. Pushed. Thanks!

@ingydotnet ingydotnet closed this Aug 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment