Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

don't include debugger gem if jruby is used since it doesn't work on jruby. #8503

Merged
merged 1 commit into from

3 participants

@siong1987

No description provided.

@bensie

:-1: It's commented out by default. Unnecessary clutter that encourages conditions for the various implementations.

@siong1987

Yes. It is commented out by default. But, users who are new to jruby will still get confused with that. Think about users who just switch to jruby from mri.

A simple change like this will help to save a lot of time for people who just switch from mri to jruby.

@carlosantoniodasilva carlosantoniodasilva merged commit 34cdaf7 into rails:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 0 deletions.
  1. +2 −0  railties/lib/rails/generators/rails/app/templates/Gemfile
View
2  railties/lib/rails/generators/rails/app/templates/Gemfile
@@ -21,5 +21,7 @@ source 'https://rubygems.org'
# Deploy with Capistrano
# gem 'capistrano', group: :development
+<% unless defined?(JRUBY_VERSION) %>
# To use debugger
# gem 'debugger'
+<% end %>
Something went wrong with that request. Please try again.