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

Add Identifier#components #677

Merged
merged 1 commit into from
Nov 14, 2015
Merged

Add Identifier#components #677

merged 1 commit into from
Nov 14, 2015

Conversation

denisdefreyne
Copy link
Member

This provides a slightly nicer-to-use alternative to identifier.to_s.split.

The return value is such that '/' + identifier.components.join('/') is identical to identifier.to_s.

See #674.

@denisdefreyne denisdefreyne added this to the 4.1.0 milestone Jul 11, 2015
@denisdefreyne denisdefreyne mentioned this pull request Jul 11, 2015
Closed
@denisdefreyne
Copy link
Member Author

Still unsure whether the components of / should be an empty list, or a list with an empty string.

denisdefreyne added a commit that referenced this pull request Nov 14, 2015
@denisdefreyne
Merge pull request #677 from nanoc/identifier-components
643f78b 
Add Identifier#components
@denisdefreyne denisdefreyne merged commit 643f78b into master Nov 14, 2015
@denisdefreyne denisdefreyne deleted the identifier-components branch November 14, 2015 09:09
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
type:enhancement ✨
Projects
None yet
Milestone
4.1.0
Development

Successfully merging this pull request may close these issues.
1 participant
@denisdefreyne