aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
* Cleanup CHANGELOGs [ci skip]Prathamesh Sonpatki2016-07-091-1/+5
| | | | | | - Cleanup Active Record CHANGELOG. - Add missing CHANGELOG for https://github.com/rails/rails/pull/25688. - Clarify that assets requests logging is suppressed.
* Merge pull request #25652 from prathamesh-sonpatki/rm-boot-railsGuillermo Iguaran2016-07-0838-46/+3
|\ | | | | Remove unused boot_rails method and it's usage
| * Remove unused boot_rails method and it's usagePrathamesh Sonpatki2016-07-0438-46/+3
| | | | | | | | | | - The `boot_rails` method from abstract_unit.rb is empty after 2abcdfd978fdcd491576a237e8c6b. - So let's remove it and its usage.
* | Merge pull request #25688 from mahnunchik/host-env-varGuillermo Iguaran2016-07-082-0/+8
|\ \ | | | | | | Use the HOST environment variable for rails server #25677
| * | Use the HOST environment variable for rails server #25677Evgeny Vlasenko2016-07-052-0/+8
| |/
* | Merge pull request #25714 from robin850/changelog-25692Rafael Mendonça França2016-07-071-0/+6
|\ \ | | | | | | | | | Add a changelog entry for #25692
| * | Add a changelog entry for #25692 [ci skip]Robin Dupret2016-07-061-0/+6
| | |
* | | Merge pull request #25730 from prathamesh-sonpatki/rm-cookie-onlyRafael França2016-07-071-0/+6
|\ \ \ | | | | | | | | No need to set `cookie_only` option from Rails
| * | | Add a test case for verifying `cookie_only` is set even if user tries to set ↵Prathamesh Sonpatki2016-07-071-0/+6
| |/ / | | | | | | | | | it false
* / / Fix typo in test caseAbhishek Jain2016-07-081-1/+1
|/ /
* | Merge pull request #25704 from ↵Rafael França2016-07-052-0/+11
|\ \ | | | | | | | | | | | | fledman/add_respond_to_missing_to_rails_application_configuration_custom implement respond_to_missing? to match method_missing
| * | implement respond_to_missing? to match method_missingDavid Feldman2016-07-052-0/+11
| | |
* | | Merge pull request #25692 from ↵Rafael França2016-07-052-0/+18
|\ \ \ | |/ / |/| | | | | | | | johnmeehan/allow-registering-directory-for-rake-notes Allow rake notes to work with other directories.
| * | Fix test to try `spec/` directory as opposed to `app/spec` which I hadJohn Meehan2016-07-051-2/+2
| | | | | | | | | | | | | | | | | | accidently set it to. Made the change and test still passes.
| * | removed extra blank lineJohn Meehan2016-07-051-1/+0
| | |
| * | Allow rake notes to work with other directories.John Meehan2016-07-052-0/+19
| |/ | | | | | | | | | | | | Additional directories can be added using SourceAnnotationExtractor::Annotation.register_directories("spec", "other_dir") Result: rake notes will now extract notes from these directories.
* | Merge pull request #25257 from ta1kt0me/inspect_initializersRichard Schneeman2016-07-053-1/+10
|\ \ | | | | | | Display railtie class name in `rails initializers`
| * | Initializer provides its context class nameta1kt0me2016-06-123-3/+10
| | |
| * | Delegate public method for raitile nameta1kt0me2016-06-112-1/+3
| | |
| * | Display railtie class name in `rails initializers`ta1kt0me2016-06-021-1/+1
| | |
* | | Point to the correct documentation [ci skip]Benjamin Quorning2016-07-051-1/+1
| | | | | | | | | | | | | | | | | | | | | The options used in this file are *not* directly described in the Rails 5.0 release notes (http://edgeguides.rubyonrails.org/5_0_release_notes.html), but instead in section 2 of the Guide for Upgrading Ruby on Rails document (http://edgeguides.rubyonrails.org/upgrading_ruby_on_rails.html).
* | | Robots.txt removed disallow all instructionsSam Sweeney2016-07-051-4/+0
| |/ |/|
* | Merge pull request #25646 from y-yagi/refer_to_rails_command_instead_of_rakeVipul A M2016-07-021-3/+3
|\ \ | | | | | | refer to rails command instead of rake in doc of `SourceAnnotationExtractor`
| * | refer to rails command instead of rake in doc of `SourceAnnotationExtractor` ↵yuuji.yaginuma2016-07-021-3/+3
| | | | | | | | | | | | [ci skip]
* | | Merge pull request #25222 from vipulnsward/25219-fix-logsMatthew Draper2016-07-023-9/+29
|\ \ \ | |/ / |/| | Clean backtrace in IRB
| * | - Added test for verifying that we treat line number indications from irb as ↵Vipul A M2016-07-011-8/+16
| | | | | | | | | | | | | | | | | | user/lib code - Cleaned up backtrace cleaner test code and removed instaces variables
| * | Fixes #25219Vipul A M2016-07-012-1/+13
| | | | | | | | | | | | Add handling of cleaning up backtrace from IRB console in case of errors
* | | fix task name to removeyuuji.yaginuma2016-07-021-1/+1
| | | | | | | | | | | | Follow up to #25431
* | | Do not run `bundle install` when generating a new plugin.Rafael Mendonça França2016-07-016-103/+112
|/ / | | | | | | | | | | Since bundler 1.12.0, the gemspec is validated so the `bundle install` command will fail just after the gem is created causing confusion to the users. This change was a bug fix to correctly validate gemspecs.
* | Merge pull request #25431 from ↵Rafael França2016-06-302-1/+10
|\ \ | | | | | | | | | | | | prathamesh-sonpatki/add-note-about-upgrade-guide-at-the-end-of-update Display link to Rails upgrade guide at the end of `rails app:update` task.
| * | Display link to Rails upgrade guide at the end of `rails app:update` task.Prathamesh Sonpatki2016-06-302-1/+10
| | | | | | | | | | | | | | | - This will nudge people in the direction of going through upgrade guides.
* | | Use sass-rails in our test suiteRafael Mendonça França2016-06-301-1/+1
| | |
* | | Merge pull request #25607 from sstephenson/turbolinks-5-finalRafael Mendonça França2016-06-301-1/+1
|\ \ \ | | | | | | | | | | | | Update to Turbolinks 5.0.0 final
| * | | Update to Turbolinks 5.0.0 finalSam Stephenson2016-06-301-1/+1
| |/ /
* / / Point to released coffee-railsRafael Mendonça França2016-06-301-1/+1
|/ /
* | Update changelog for #25351Kevin McPhillips2016-06-221-0/+5
| |
* | Initialize the configuration with a valueRafael Mendonça França2016-06-224-10/+2
| | | | | | | | | | | | | | This will fix the failures fixed by 41488adbb89a09fa96188486f8414842ff28c848 and keep this new option that is semi-private outside of the generated configurations
* | Fix failing railties testseileencodes2016-06-223-0/+9
| | | | | | | | | | Railties tests didn't know about the new config option added in 80b416f so it needed to be added to the railties generators for configs.
* | Add option to enable dependency loading in productionRafael Mendonça França2016-06-221-1/+1
| | | | | | | | | | | | | | | | This will make easier to applications that rely on having const_missing hooks in production upgrade to Rails 5. This option is going to be remove in the future and the default behavior will be to disable the dependency loading.
* | Fix rails/info routes for apps with globbing routeNicholas Firth-McCoy2016-06-203-1/+27
| | | | | | | | | | | | | | | | The /rails/info routes were inaccessible in apps with a catch-all globbing route, as they were being appended after the globbing route and would never be matched. See also ccc3ddb7762bae0df7e2f8d643b19b6a4769d5be.
* | No need to have conditional based on api options in assetsPrathamesh Sonpatki2016-06-201-2/+0
| | | | | | | | | | | | - As assets are removed for API only apps anyways, so we don't need any conditional upfront. - assets are removed for API apps here - https://github.com/rails/rails/blob/94ef224aa61ace3fa643eab161ff9056b7d90a62/railties/lib/rails/generators/rails/app/app_generator.rb#L266-L273.
* | Fix failing railties testsPrathamesh Sonpatki2016-06-181-2/+2
| | | | | | | | | | | | | | | | | | | | - Railties tests related to fetching asset URL started failing after the release of sprockets 3.6.1 on Travis. - This was due to the change in https://github.com/rails/sprockets/pull/311/files which changed the logic in `concat_javascript_sources` to add `;` at the end of file if the source did not end with semicolon. - Bumped up sprockets minor version and fixed the failing tests.
* | Move quiet option to development.rbRafael Mendonça França2016-06-162-3/+3
| | | | | | | | | | This option is used in a initializer hook that runs before the config/initializers files so it will not work if it is set there.
* | Merge pull request #25390 from gsamokovarov/abstact-unit-presence-requireRafael França2016-06-131-0/+1
|\ \ | | | | | | Require "active_support/core_ext/object/blank" for railties abstract unit
| * | Require "active_support/core_ext/object/blank" for railties abstract unitGenadi Samokovarov2016-06-131-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | In #25380 I moved an `active_support/core_ext/object/blank` in the `activesupport/lib/active_support/testing/assertions.rb`, however it caused a chain reaction, resulting in failing railties tests. Moving the require to `railties/test/isolation/abstract_unit.rb` and `activesupport/test/abstract_unit.rb`.
* | | Merge pull request #25319 from morgoth/generate-application-record-firstRafael França2016-06-134-18/+18
|\ \ \ | |/ / |/| | Generate application_record.rb file before model file
| * | Generate application_record.rb file before model fileWojciech Wnętrzak2016-06-084-18/+18
| | | | | | | | | | | | | | | | | | Previously model file was generated first, which resulted in inheriting from `ActiveRecord::Base`, but since application_record.rb is generated as well, it should already be used.
* | | Merge pull request #25004 from ↵Eileen M. Uchitelle2016-06-125-0/+44
|\ \ \ | | | | | | | | | | | | | | | | y-yagi/generate_mailer_layout_files_if_it_does_not_already_exist generate mailer layout files if it does not already exist
| * | | generate mailer layout files if it does not already existyuuji.yaginuma2016-05-155-0/+44
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, if `ApplicationMailer` does not exist, it is generated when run the mailer generator, but layouts files does not generate. However, because it uses the layouts in `ApplicationMailer`, layouts are required. Follow up to #24161
* | | | Add config.assets.quiet = true as default valueKevin McPhillips2016-06-101-0/+3
| | | |