aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
* Included rails-dom-testing in rails_info_controller_test.rbTimm2014-06-161-0/+3
|
* [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
| | | |
* | | | Generate the cookies_serializer configuration with :marshal when upgradingRafael Mendonça França2014-05-162-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We were generating this configuration with :json what may break applications upgrating from Rails 4.0-. Now we always generate with a safer value and people is free to choose hybrid if they want. We don't generate with hybrid by default because there are some cases that can fail. These cases are explained on the upgrading guide.
* | | | Ok. Enough models for today :rage:Rafael Mendonça França2014-05-121-3/+6
| | | |
* | | | This is also a modelRafael Mendonça França2014-05-121-1/+1
| | | |
* | | | This model is a model so it should behaves like oneRafael Mendonça França2014-05-121-7/+3
| | | |
* | | | Replace :base with :in [ci skip]Marcio Junior2014-05-111-2/+2
| | | |
* | | | Change Gemfile’s double quotes in plugin generatorJosemar Luedke2014-05-102-7/+7
| |/ / |/| | | | | | | | | | | Since the generated application’s Gemfile is using single quotation marks, it seems fair to use it on plugin’s Gemfile as well.
* | | Merge pull request #15015 from pothibo/database-configurationRafael Mendonça França2014-05-073-1/+22
|\ \ \ | | | | | | | | Use Rails::Paths::Path#existent in database_configuration
| * | | Use Rails::Paths::Path#existent in database_configurationPier-Olivier Thibault2014-05-073-1/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Database configuration was trying to load the first path that config.paths['config/database'] was returning even if the path didn't exist in the filesystem. Because Rails::Paths::Path has the possibility to return more than 1 path (as an array), database_configuration should filter down the paths to the existing one and then load the first one. This would make it possible to move the database.yml file and add the new path to paths['config/database'] and still load the configurations.
* | | | Return a non zero code when db has never been setup on statusPaul B2014-05-071-1/+1
|/ / /