Skip to content
This repository has been archived by the owner on Sep 21, 2022. It is now read-only.

Fix quoting in check_mount() #146

Closed
wants to merge 1 commit into from

Conversation

alexjh
Copy link
Contributor

@alexjh alexjh commented Jan 20, 2017

No description provided.

@cfdreddbot
Copy link

Hey alexjh!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@cf-gitbot
Copy link
Collaborator

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/138063035

The labels on this github issue will be updated when the story is started.

@robdimsdale
Copy link
Member

@alexjh thanks for this. To the best of our ability, we cannot see where this method is used, so rather than merging this PR we'd rather remove this function entirely.

Before we do that, can you tell us if you actually use this function somewhere, or were you just cleaning up quoting because you saw it was potentially a problem?

@alexjh
Copy link
Contributor Author

alexjh commented Jan 26, 2017

This was just a drive-by clean up that I noticed when tracking down something else. :) Removing the function would be fine with me.

@robdimsdale
Copy link
Member

Great, thanks. We will remove this function instead of merging this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants