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 -j (--javascript) option from `rails new` #28546

Merged
merged 1 commit into from Mar 27, 2017

Conversation

Projects
None yet
5 participants
@claudiob
Member

claudiob commented Mar 23, 2017

The "-j" option was added 5 years ago (d9c39c3)
when we wanted to support prototype-rails and jquery-rails.
Prototype is not as popular and jQuery is not a requirement anymore.
Still the "-j" option can be used to install any gem that ends in "-rails".

This "might" open security issues and does not bring great benefits anymore.

If you know which "-rails"-ending gem you want to install, you can manually
add it to the Gemfile just like any other gem.

Remove -j (--javascript) option from `rails new`
The "-j" option was added 5 years ago (d9c39c3)
when we wanted to support prototype-rails and jquery-rails.
Prototype is not as popular and jQuery is not a requirement anymore.
Still the "-j" option can be used to install *any* gem that ends in "-rails".

This "might" open security issues and does not bring great benefits anymore.

If you know which "-rails"-ending gem you want to install, you can manually
add it to the Gemfile just like any other gem.

@claudiob claudiob added the railties label Mar 23, 2017

@claudiob claudiob added this to the 5.2.0 milestone Mar 23, 2017

@claudiob

This comment has been minimized.

Show comment
Hide comment
@claudiob

claudiob Mar 23, 2017

Member

My opinion is that we don't need a full deprecation warning cycle for this change, since it only affects generating new apps, not existing apps. Let me know if that's not the case.

I looked for -j and --javascript elsewhere in the code and in the guides and did not find any other reference… let me know if I forgot anything!

Member

claudiob commented Mar 23, 2017

My opinion is that we don't need a full deprecation warning cycle for this change, since it only affects generating new apps, not existing apps. Let me know if that's not the case.

I looked for -j and --javascript elsewhere in the code and in the guides and did not find any other reference… let me know if I forgot anything!

@kaspth

This comment has been minimized.

Show comment
Hide comment
@kaspth

kaspth Mar 23, 2017

Member

It's possible for people to have an app template that includes the option. Perhaps a .railsrc too.

Perhaps it's best to deprecate.

Member

kaspth commented Mar 23, 2017

It's possible for people to have an app template that includes the option. Perhaps a .railsrc too.

Perhaps it's best to deprecate.

@sunnyrjuneja

This comment has been minimized.

Show comment
Hide comment
@sunnyrjuneja

sunnyrjuneja Mar 27, 2017

I think this makes a lot of sense. It would also alleviate the confusion in my issue created here: #28512

sunnyrjuneja commented Mar 27, 2017

I think this makes a lot of sense. It would also alleviate the confusion in my issue created here: #28512

@rafaelfranca

This comment has been minimized.

Show comment
Hide comment
@rafaelfranca

rafaelfranca Mar 27, 2017

Member

If they have in the .railsrc it will just be ignored.

Member

rafaelfranca commented Mar 27, 2017

If they have in the .railsrc it will just be ignored.

@rafaelfranca rafaelfranca modified the milestones: 5.1.0, 5.2.0 Mar 27, 2017

@rafaelfranca

This comment has been minimized.

Show comment
Hide comment
@rafaelfranca

rafaelfranca Mar 27, 2017

Member

I'll include this in 5.1 since we already include rails-ujs.

Member

rafaelfranca commented Mar 27, 2017

I'll include this in 5.1 since we already include rails-ujs.

rafaelfranca added a commit that referenced this pull request Mar 27, 2017

Merge pull request #28546 from claudiob/drop-j-option
Remove -j (--javascript) option from `rails new`

@rafaelfranca rafaelfranca merged commit 4219806 into rails:master Mar 27, 2017

2 checks passed

codeclimate no new or fixed issues
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@rafaelfranca

This comment has been minimized.

Show comment
Hide comment
@rafaelfranca

rafaelfranca Mar 27, 2017

Member

Backported in 0b0255d

Member

rafaelfranca commented Mar 27, 2017

Backported in 0b0255d

rafaelfranca added a commit that referenced this pull request Mar 27, 2017

Merge pull request #28546 from claudiob/drop-j-option
Remove -j (--javascript) option from `rails new`

@claudiob claudiob deleted the claudiob:drop-j-option branch Mar 28, 2017

y-yagi added a commit to y-yagi/rails that referenced this pull request Mar 28, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment