Skip to content

Conversation

jrafanie
Copy link
Member

Even though we don't use the added features of Pathnames here, we should accept them and convert to a new String.

Later, we can perhaps clean up some of the String processing by fully supporting pathnames and in fact, convert any provided string paths to pathnames.

@jrafanie
Copy link
Member Author

ping @brandondunne

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling eb7cf3e on jrafanie:support_pathname_for_logical_volume_name into 7d73190 on ManageIQ:master.

bdunne added a commit that referenced this pull request Nov 15, 2013
…ume_name

Support a Pathname lv 'name' by converting it to a new String.
@bdunne bdunne merged commit 70ca0dc into ManageIQ:master Nov 15, 2013
@jrafanie jrafanie deleted the support_pathname_for_logical_volume_name branch November 15, 2013 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants