Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[#4403] To be consistent with MRI, File.exists? for <path to file> end… #4422

Closed
wants to merge 1 commit into from
Closed

[#4403] To be consistent with MRI, File.exists? for <path to file> end… #4422

wants to merge 1 commit into from

Conversation

sumitmah
Copy link

@sumitmah sumitmah commented Jan 6, 2017

…ing with / should return false

@sumitmah sumitmah changed the title [4403] To be consistent with MRI, File.exists? for <path to file> end… [#4403] To be consistent with MRI, File.exists? for <path to file> end… Jan 6, 2017
@kares
Copy link
Member

kares commented Jan 23, 2017

Thanks Sumit, there are a few regressions thus this won't do as is. Likely needs work for the corner cases.

@kares
Copy link
Member

kares commented Jan 10, 2018

closing since the change had regression and the author did not get back to it

@kares kares closed this Jan 10, 2018
@kares kares added this to the Invalid or Duplicate milestone Jan 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
in progress
Projects
None yet
Milestone
Invalid or Duplicate
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@sumitmah @kares