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.

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

Change Journey::Route#verb to return string instead of regexp. #21849

Merged
merged 1 commit into from
Oct 12, 2015

Conversation

yui-knk
Copy link
Contributor

@yui-knk yui-knk commented Oct 3, 2015

By this commit
Journey::Route#verb need not to return verb as regexp.
The returned value is used by inspector, so change it to be a string.

Add inspect_with_multiple_verbs test case to keep the behavior of
inspector correctly.

By [this commit](rails@0b476de)
`Journey::Route#verb` need not to return verb as regexp.
The returned value is used by inspector, so change it to be a string.

Add inspect_with_multiple_verbs test case to keep the behavior of
inspector correctly.
@rails-bot
Copy link

r? @eileencodes

(@rails-bot has picked a reviewer for you, use r? to override)

@yui-knk
Copy link
Contributor Author

yui-knk commented Oct 3, 2015

r? @tenderlove

@rails-bot rails-bot assigned tenderlove and unassigned eileencodes Oct 3, 2015
@yui-knk
Copy link
Contributor Author

yui-knk commented Oct 3, 2015

When this PR is merged, I will close #20787 :)

pixeltrix added a commit that referenced this pull request Oct 12, 2015
Change `Journey::Route#verb` to return string instead of regexp.
@pixeltrix pixeltrix merged commit f39ab9f into rails:master Oct 12, 2015
@pixeltrix
Copy link
Contributor

@yui-knk thanks!

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.

None yet

5 participants