aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
...
* | | :scissors:Rafael Mendonça França2014-06-171-8/+8
| | |
* | | Add a generic --skip-gems options to generatorRafael Mendonça França2014-06-173-10/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also remove --skip-turbolinks. This option is useful if users want to remove some gems like jbuilder, turbolinks, coffee-rails, etc that don't have specific options on the generator. rails new my_app --skip-gems turbolinks coffee-rails
* | | Merge pull request #15716 from schuetzm/skip-turbolinksRafael Mendonça França2014-06-172-2/+21
|\ \ \ | | | | | | | | Add an option to skip installation of Turbolinks
| * | | Add an option to skip installation of TurbolinksMarc Schütz2014-06-142-2/+21
| | | |
* | | | allow preview interceptors to be registered through `config.action_mailer`.Yves Senn2014-06-151-0/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was partially broken because `preview_interceptors=` just assigned the raw values, whithout going through `register_preview_interceptor`. Now the Action Mailer railtie takes care of the `preview_interceptors` option. This commit is a partial revert of: Revert "Merge pull request #15739 from y-yagi/correct_doc_for_action_mailer_base" This reverts commit a15704d7f35f17d34d0118546799141d6f853656, reversing changes made to 1bd12a8609d275ad75fcc4b622ca4f5b32dc76be. /cc @kuldeepaggarwal @y-yagi
* | | | [ci-skip] correct wikipedia MVC linkAditya Kapoor2014-06-141-1/+1
|/ / /
* | | Merge pull request #15394 from ↵Yves Senn2014-06-121-1/+90
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | morgoth/fix-automatic-maintaining-test-schema-for-sql-format ActiveRecord::Migration.maintain_test_schema! doesn't work with structure.sql Conflicts: activerecord/CHANGELOG.md
| * | | Fixed automatic maintaining test schema to properly handle sql structure ↵Wojciech Wnętrzak2014-06-121-2/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | schema format. Additionally: * It changes `purge` task on `sqlite3` adapter to recreate database file, to be consistent with other adapters. * Adds `purge` step when loading from `schema.rb`
| * | | Added missing test case for migrations when using sql structureWojciech Wnętrzak2014-06-011-1/+40
| | | |
* | | | remove unnused method from testFabian Rodriguez2014-06-111-10/+0
| | | |
* | | | [ci skip] Fix capitalizationAkshay Vishnoi2014-06-071-4/+4
| | | |
* | | | remove rubyforge.org that was shut down [ci skip]Gaurav Sharma2014-06-062-4/+2
| | | |
* | | | Emit suggested generator names when not foundschneems2014-06-043-9/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When someone types in a generator command it currently outputs all generators. Instead we can attempt to find a subtle mis-spelling by running all generator names through a levenshtein_distance algorithm provided by rubygems. So now a failure looks like this: ```ruby $ rails generate migratioooons Could not find generator 'migratioooons'. Maybe you meant 'migration' or 'integration_test' or 'generator' Run `rails generate --help` for more options. ``` If the suggestions are bad we leave the user with the hint to run `rails generate --help` to see all commands.
* | | | File.exist? instead of File.exists? in bin/setupDavid Jones2014-06-021-1/+1
| | | | | | | | | | | | | | | | | | | | File.exists? is deprecated in Ruby 2.1+ https://github.com/ruby/ruby/blob/v2_1_2/file.c#L1413
* | | | Feature requests should be made on the mailing list, not submitted toZachary Scott2014-06-011-1/+5
| | | | | | | | | | | | | | | | the issue tracker. See also #15455 [ci skip]
* | | | Merge pull request #14881 from robin850/patch-19Yves Senn2014-06-011-5/+8
|\ \ \ \ | |/ / / |/| | | Swallow bundler exceptions when running tests
| * | | Swallow bundler exceptions when running testsRobin Dupret2014-04-271-5/+8
| | | | | | | | | | | | | | | | Let's avoid useless output when testing stuff related to Bundler.
* | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2014-06-011-0/+5
|\ \ \ \
| * | | | document generators hidden_namespacesschneems2014-05-291-0/+5
| | | | |
* | | | | bin/setup uses `bundle check || bundle install` to improve performance.Yves Senn2014-05-301-1/+1
| | | | |
* | | | | `bin/setup` script to bootstrap applications.Yves Senn2014-05-303-0/+33
| | | | |
* | | | | Revert "Merge pull request #12778 from etehtsea/rake-default-fix"Yves Senn2014-05-282-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit ec0a2b57f67c9153cb5f7bbe2a3f66d13fe64bdd, reversing changes made to a8bd7b16260c217290ef8fe3a166fcceac442c62. This reverts commit 555ec36522011862c03b483c53be32410594a51e This reverts commit 555ec36522011862c03b483c53be32410594a51e By default, Rails uses the `default` Rake task to run all tests. This commit changed the environment of the default task to `development`. This clears the development database and has other negative consequences.
* | | | | Merge pull request #15269 from arunagw/aa-fixes-8930Rafael Mendonça França2014-05-272-0/+41
|\ \ \ \ \ | | | | | | | | | | | | rake railties:install:migrations respects the order of railties
| * | | | | rake railties:install:migrations respects the order of railtiesArun Agrawal2014-05-272-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | This PR fixes #8930 and some stuff from #8985
* | | | | | Deprecate all *_filter callbacks in favor of *_action callbacksRafael Mendonça França2014-05-272-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the continuation of the work started at 9d62e04838f01f5589fa50b0baa480d60c815e2c
* | | | | | Merge pull request #15327 from alexbel/replace_double_quotes_with_single_quotesYves Senn2014-05-273-11/+32
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Replace double quotes with single quotes while adding an entry into Gemfile
| * | | | | | Replace double quotes with single quotes while adding an entry into Gemfilealexbel2014-05-263-11/+25
|/ / / / / /
* | | | | | Merge pull request #15333 from ShunsukeAida/update_ruby_versionRafael Mendonça França2014-05-261-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | update the recommended ruby version.
| * | | | | | update the recommended ruby versionShunsukeAida2014-05-271-1/+1
| | | | | | |
* | | | | | | Merge pull request #15294 from schneems/schneems/generator-output-in-error2Rafael Mendonça França2014-05-262-14/+20
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | print generators on failed generate
| * | | | | | print generators on failed generateschneems2014-05-262-14/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let's say we just ran: ``` $ rails g migrate add_click_to_issue_assignment ``` We will get an error that looks like: ``` Could not find generator migrate. ``` This patch adds all existing migrations to the output to make it easier for a developer to find a valid migration. ``` Could not find generator "migrate". Please select a valid generator: Rails: assets controller generator helper integration_test mailer migration model resource scaffold scaffold_controller task ``` It would be nice to do some spelling detection and suggest alternatives, but for now this should help.
* | | | | | | simple solution that brings rake stats task to every kind of rails projectAndriy Tyurnikov2014-05-261-18/+22
| | | | | | |
* | | | | | | Merge pull request #15305 from tgxworld/remove_unnecessary_requireGuillermo Iguaran2014-05-242-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Remove unnecessary require of Minitest.
| * | | | | | | Remove unnecessary require of Minitest.Guo Xiang Tan2014-05-242-2/+2
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | Minitest has already been required when calling Minitest.autorun.
* | | | | | | integration tests do not need to load fixtures by handXavier Noria2014-05-251-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I don't know when this changed, checked back up to 3.2 and integration tests instantiate fixtures just fine.
* | | | | | | Merge pull request #15299 from andriytyurnikov/engine_friendly_rake_statsRafael Mendonça França2014-05-242-19/+24
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | rake stats for engines
| * | | | | | | rake stats for enginesAndriy Tyurnikov2014-05-242-19/+24
| |/ / / / / /
* | | | | | | Merge pull request #15298 from ShunsukeAida/fix_capitalizationRafael Mendonça França2014-05-241-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | fixed capitalization [ci skip]
| * | | | | | | fixed capitalizationShunsukeAida2014-05-251-1/+1
| | |/ / / / / | |/| | | | |
* | | | | | | Add FIXME note about the thor bugRafael Mendonça França2014-05-241-0/+2
| | | | | | |
* | | | | | | Merge pull request #9083 from bronson/patch-1Rafael Mendonça França2014-05-241-1/+4
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | avoid using alias in generators
| * | | | | | avoid using alias in generatorsScott Bronson2013-01-251-1/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ruby's alias produces public methods, causing a spurious Thor task to be created.  For example, this is the reason MigrationGenerator currently has two tasks:     > ActiveRecord::Generators::MigrationGenerator.all_tasks.keys     => ["singular_name", "create_migration_file"] singular_name was meant to be an attribute, not a task.  Because it's public, it gets called as a task every time the generator is invoked. The fix is to ensure all generator methods have the correct visibility.
* | | | | | | Revert "Merge pull request #15293 from ↵Yves Senn2014-05-241-16/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | schneems/schneems/generator-output-in-error" This reverts commit 7466d6a9f989a1a8f90899b79b34f1152e440b7c, reversing changes made to b850c4c540e8f0c15742f67c532f521a4a123655. This was an accidental merge. A new PR will follow. /cc @schneems
* | | | | | | Merge pull request #15293 from schneems/schneems/generator-output-in-errorRichard Schneeman2014-05-241-12/+16
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | print generators on failed generate
| * | | | | | | print generators on failed generateschneems2014-05-241-12/+16
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Let's say we just ran: ``` $ rails g migrate add_click_to_issue_assignment ``` We will get an error that looks like: ``` Could not find generator migrate. ``` This patch adds all existing migrations to the output to make it easier for a developer to find a valid migration. ``` Could not find generator "migrate". Please select a valid generator: Rails: assets controller generator helper integration_test mailer migration model resource scaffold scaffold_controller task ``` It would be nice to do some spelling detection and suggest alternatives, but for now this should help.
* / | | | | | Future port c8ddb61Zachary Scott2014-05-231-2/+1
|/ / / / / /
* | | | | / Default config.assets.digests to true in developmentDan Kang2014-05-174-14/+20
| |_|_|_|/ |/| | | |
* | | | | New applications should be created with json serializerRafael Mendonça França2014-05-172-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fix introduced at 75ba7425f6d235f254c00a45a58f42e88f80301a were missing this case. To fix the original issue mentioned at that commit the correct is to remove update_config_files from the command list since we should not run both config_files command when creating a new application. Tests were added to make sure everything is working fine.
* | | | | Fix app generator test hanging.Christian Wesselhoeft2014-05-172-6/+2
| | | | |
* | | | | foo :bomb:Rafael Mendonça França2014-05-161-2/+5
| | | | |