Skip to content

Commit

Permalink
Merge pull request rails#54313 from eelcoj/update-minimal-flag
Browse files Browse the repository at this point in the history
Extend --minimal option
  • Loading branch information
rafaelfranca authored Jan 24, 2025
2 parents 31f8a6b + dad0763 commit cd171d6
Show file tree
Hide file tree
Showing 3 changed files with 21 additions and 0 deletions.
7 changes: 7 additions & 0 deletions railties/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,10 @@
* Update `rails new --minimal` option

Extend the `--minimal` flag to exlcude recently added features:
`skip_brakeman`, `skip_ci`, `skip_docker`, `skip_kamal`, `skip_rubocop`, `skip_solid` and `skip_thruster`.

*eelcoj*

* Add `application-name` metadata to application layout

The following metatag will be added to `app/views/layouts/application.html.erb`
Expand Down
7 changes: 7 additions & 0 deletions railties/lib/rails/generators/rails/app/app_generator.rb
Original file line number Diff line number Diff line change
Expand Up @@ -305,11 +305,18 @@ class AppGenerator < AppBase
:skip_active_job,
:skip_active_storage,
:skip_bootsnap,
:skip_brakeman,
:skip_ci,
:skip_dev_gems,
:skip_docker,
:skip_hotwire,
:skip_javascript,
:skip_jbuilder,
:skip_kamal,
:skip_rubocop,
:skip_solid,
:skip_system_test,
:skip_thruster
],
api: [
:skip_asset_pipeline,
Expand Down
7 changes: 7 additions & 0 deletions railties/test/generators/app_generator_test.rb
Original file line number Diff line number Diff line change
Expand Up @@ -1265,11 +1265,18 @@ def test_minimal_rails_app
assert_option :skip_active_job
assert_option :skip_active_storage
assert_option :skip_bootsnap
assert_option :skip_brakeman
assert_option :skip_ci
assert_option :skip_dev_gems
assert_option :skip_docker
assert_option :skip_hotwire
assert_option :skip_javascript
assert_option :skip_jbuilder
assert_option :skip_kamal
assert_option :skip_rubocop
assert_option :skip_solid
assert_option :skip_system_test
assert_option :skip_thruster
end

def test_minimal_rails_app_with_no_skip_implied_option
Expand Down

0 comments on commit cd171d6

Please sign in to comment.