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

Point to the correct documentation #25696

Merged

Conversation

bquorning
Copy link
Contributor

The options used in this file are not directly described in the Rails 5.0 release notes, but instead in section 2 of the Guide for Upgrading Ruby on Rails document.

cc/ @prathamesh-sonpatki @kaspth

Should I open a PR for backporting to 5-0-0-stable?

The options used in this file are *not* directly described in the Rails 5.0
release notes (http://edgeguides.rubyonrails.org/5_0_release_notes.html), but
instead in section 2 of the Guide for Upgrading Ruby on Rails document
(http://edgeguides.rubyonrails.org/upgrading_ruby_on_rails.html).
@rails-bot
Copy link

r? @kaspth

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

@prathamesh-sonpatki prathamesh-sonpatki merged commit 087ca9a into rails:master Jul 5, 2016
@prathamesh-sonpatki
Copy link
Member

Thanks @bquorning! I think we can backport directly to 5-0-stable the merge commit without going through PR. That's how we have been doing it for other commits which need to be backported.

@kaspth can do that :) I can't directly push to 5-0-stable, so tagging this with needs backport 😄

@vipulnsward
Copy link
Member

This should not have [ci skip] . Its not a doc change.

@prathamesh-sonpatki
Copy link
Member

My bad. @kaspth let me know if we should revert it.

@bquorning bquorning deleted the link-to-the-right-documentation branch July 5, 2016 10:00
@bquorning
Copy link
Contributor Author

This should not have [ci skip]. Its not a doc change.

Oops, you’re right. It does, however, only change a comment.

I’ll be more careful with the ci skip in the future.

kaspth pushed a commit to kaspth/rails that referenced this pull request Jul 5, 2016
…entation

Point to the correct documentation
@kaspth
Copy link
Contributor

kaspth commented Jul 5, 2016

@prathamesh-sonpatki only owners can push to 5-0-stable, so it's fine to create a backport PR, which I've done in #25705.

kaspth added a commit that referenced this pull request Jul 5, 2016
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

6 participants