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

Use Pathname#to_s instead of Pathname#to_path #1730

Merged
merged 1 commit into from Nov 18, 2013

Conversation

Projects
None yet
3 participants
@parkr
Member

parkr commented Nov 16, 2013

Should fix #1723. Works locally on v1.8.7-p371

@parkr

This comment has been minimized.

Member

parkr commented Nov 17, 2013

@mattr-, what do you think?

@mattr-

This comment has been minimized.

Member

mattr- commented Nov 18, 2013

👍 LGTM!

mattr- added a commit that referenced this pull request Nov 18, 2013

Merge pull request #1730 from mojombo/pathname-to-s
Use Pathname#to_s instead of Pathname#to_path

@mattr- mattr- merged commit c120b96 into master Nov 18, 2013

1 check passed

default The Travis CI build passed
Details

mattr- added a commit that referenced this pull request Nov 18, 2013

@parkr parkr deleted the pathname-to-s branch Nov 18, 2013

csim added a commit to csim/jekyll that referenced this pull request Nov 22, 2013

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

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