Fix test failures in t/pod.t #2

Merged
merged 1 commit into from Sep 14, 2013

Conversation

Projects
None yet
5 participants
@chromatic
Contributor

chromatic commented May 29, 2012

The culprit was a spurious Unicode space.

@tokuhirom

This comment has been minimized.

Show comment Hide comment
@tokuhirom

tokuhirom Oct 25, 2012

+1

+1

@tokuhirom

This comment has been minimized.

Show comment Hide comment
@tokuhirom

tokuhirom Oct 25, 2012

And move this test to xt/ is more better.

And move this test to xt/ is more better.

@rjbs

This comment has been minimized.

Show comment Hide comment
@rjbs

rjbs Jan 5, 2013

I am also hoping to see this merged and released!

rjbs commented Jan 5, 2013

I am also hoping to see this merged and released!

@rsrchboy

This comment has been minimized.

Show comment Hide comment
@rsrchboy

rsrchboy Sep 8, 2013

+1 -- very frustrating :)

rsrchboy commented Sep 8, 2013

+1 -- very frustrating :)

@sukria

This comment has been minimized.

Show comment Hide comment
@sukria

sukria Sep 9, 2013

Owner

Going to merge that soon! Thanks :)

Owner

sukria commented Sep 9, 2013

Going to merge that soon! Thanks :)

@rjbs

This comment has been minimized.

Show comment Hide comment
@rjbs

rjbs Sep 9, 2013

I should've also noted that I also agree that pod.t should not be running for every user installing the library. Better to only run if AUTHOR_TESTING is set to put the test in xt, or something like that.

Anyway, thanks!

rjbs commented Sep 9, 2013

I should've also noted that I also agree that pod.t should not be running for every user installing the library. Better to only run if AUTHOR_TESTING is set to put the test in xt, or something like that.

Anyway, thanks!

sukria added a commit that referenced this pull request Sep 14, 2013

Merge pull request #2 from chromatic/master
Fix test failures in t/pod.t

@sukria sukria merged commit 01e0c84 into sukria:master Sep 14, 2013

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