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

Remove gem check in guides/rails_guides.rb #19240

Merged
merged 1 commit into from Mar 18, 2015

Conversation

yui-knk
Copy link
Contributor

@yui-knk yui-knk commented Mar 7, 2015

Now rails/Gemfile includes nokogiri and redcarpet,
so remove gem check from guides/rails_guides.rb

Now rails/Gemfile includes nokogiri and redcarpet,
so remove gem check from guides/rails_guides.rb
@schneems
Copy link
Member

schneems commented Mar 7, 2015

Build is failing

@yui-knk
Copy link
Contributor Author

yui-knk commented Mar 7, 2015

guides/ has no test so I think this PR never breaks tests.
Maybe https://travis-ci.org/rails/rails/builds/53372621 is effected.

@senny
Copy link
Member

senny commented Mar 18, 2015

@yui-knk thank you. 💛

@senny senny merged commit 2a30762 into rails:master Mar 18, 2015
@senny
Copy link
Member

senny commented Mar 18, 2015

@yui-knk as a side note. These checks were not because our own Gemfile didn't include the gems but because this file was required by user applications. See cd7cc52

senny added a commit that referenced this pull request Mar 18, 2015
Remove gem check in guides/rails_guides.rb

Conflicts:
	guides/rails_guides.rb
@yui-knk yui-knk deleted the refactor/guide_rb branch November 2, 2015 05:43
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

3 participants