aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
* Remove bin/yarn if yarn is skipped, tidy up testsPrathamesh Sonpatki2017-01-012-2/+9
| | | | | | - No need to remove bin/yarn separately for API only apps because :skip_yarn is set to true for API only apps. - Added a test for :skip_yarn config.
* Merge pull request #27535 from y-yagi/add_test_script_for_railtiesKasper Timm Hansen2017-01-011-0/+4
|\ | | | | add `bin/test` script for railties
| * add `bin/test` script for railtiesyuuji.yaginuma2017-01-011-0/+4
| | | | | | | | | | The railties test does not require specific setup performed in the rake-tasks, so we can use test runner as well as other components.
* | remove unnecessary `remove_file`yuuji.yaginuma2017-01-011-1/+0
|/ | | | | | It deletes the `app/mailers` directory itself, it is not necessary to delete the file individually. https://github.com/rails/rails/blob/master/railties/lib/rails/generators/rails/app/app_generator.rb#L329
* Match Thor's `desc` signature.Kasper Timm Hansen2016-12-311-1/+1
| | | | It can also take an options hash.
* Prevent command name being printed twice.Kasper Timm Hansen2016-12-311-0/+8
| | | | | | | | | Thor would inadvertantly duplicate the command usage because of the help method in a command class. Fixes #26664. [ Yuji Yaginuma & Kasper Timm Hansen ]
* Bump license years for 2017Jon Moss2016-12-311-1/+1
| | | | | | | | Per https://www.timeanddate.com/counters/firstnewyear.html, it's already 2017 in a lot of places, so we should bump the Rails license years to 2017. [ci skip]
* remove `skip_bundle` option from plugin generatoryuuji.yaginuma2016-12-312-3/+3
| | | | | | | Because `bundle install` is not executed regardless of whether the option is specified or not. Ref: fbd1e98cf983572ca9884f17f933ffe92833632a
* Enforce middleware ordering with a test, instead of commentsMatthew Draper2016-12-312-6/+35
| | | | | | | | | We want the actual order to be very predictable, so it's rightly defined in code -- not with an on-the-fly tsort. But we can do the tsort here, and then verify that it matches the implemented ordering. This way we don't leave future readers guessing which parts of the ordering are deliberate and which are arbitrary.
* Merge pull request #27515 from ↵Matthew Draper2016-12-312-3/+4
|\ | | | | | | | | | | kbrock/fix_log_remote_ip_before_dispatcher_ips_settings Allow log remote ip addres when config.action_dispatch.trusted_proxie…
| * Allow log remote ip addres when config.action_dispatch.trusted_proxies passedLeonid Batizhevsky2016-12-302-2/+3
|/
* Remove needless `ActiveSupport::Deprecation.silence`Ryuta Kamizono2016-12-311-3/+1
|
* `#tables` and `#table_exists?` and returns only tables and not viewsRafael Mendonça França2016-12-291-5/+3
|
* Remove deprecated support to passing a column to #quoteRafael Mendonça França2016-12-292-18/+2
|
* Merge branch 'master' into fix_26964Kasper Timm Hansen2016-12-2960-522/+551
|\
| * Merge branch 'master' into clear_all_environments_log_by_defaultRafael França2016-12-29100-588/+816
| |\
| | * Fix Rubocop violations and fix documentation visibilityRafael Mendonça França2016-12-281-38/+38
| | | | | | | | | | | | | | | | | | Some methods were added to public API in 5b14129d8d4ad302b4e11df6bd5c7891b75f393c and they should be not part of the public API.
| | * Add backticks to show shell commandJon Moss2016-12-271-2/+2
| | | | | | | | | | | | [ci skip]
| | * Remove random extra spaces from Action Pack and Railties CHANGELOG.mdJon Moss2016-12-271-2/+2
| | | | | | | | | | | | [ci skip]
| | * assert_equal takes expectation firstAkira Matsuda2016-12-261-1/+1
| | |
| | * Correct indent-accounting in controller route generationMatthew Draper2016-12-252-12/+15
| | | | | | | | | | | | | | | | | | Fixes #27447 [Matthew Draper & Yuuji Yaginuma]
| | * "Use assert_nil if expecting nil. This will fail in minitest 6."Akira Matsuda2016-12-255-8/+8
| | |
| | * Privatize unneededly protected methods in RailtiesAkira Matsuda2016-12-2525-135/+136
| | |
| | * Privatize some methods that were originally protectedAkira Matsuda2016-12-251-25/+27
| | |
| | * Prefer class << self; def over def self.Akira Matsuda2016-12-251-200/+200
| | |
| | * `protected` here doesn't protect anythingAkira Matsuda2016-12-251-26/+25
| | | | | | | | | | | | there aren't any instance method defined in this class
| | * Privatize unneededly protected methods in Railties testsAkira Matsuda2016-12-246-6/+6
| | |
| | * Describe what we are protectingAkira Matsuda2016-12-232-0/+4
| | |
| | * quiet generators log in testyuuji.yaginuma2016-12-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This quiet the following log. ``` create app/mailers/notifier_mailer.rb invoke erb create app/views/notifier_mailer identical app/views/layouts/mailer.text.erb identical app/views/layouts/mailer.html.erb create app/views/notifier_mailer/foo.text.erb create app/views/notifier_mailer/foo.html.erb invoke test_unit create test/mailers/notifier_mailer_test.rb create test/mailers/previews/notifier_mailer_preview.rb ```
| | * Limit length of secret being passedJon Moss2016-12-201-3/+3
| | | | | | | | | | | | Very similar to PR #25758, see more in depth reasoning there.
| | * `ARGV.shift` before calling Rails generatorsJon Moss2016-12-172-0/+17
| | | | | | | | | | | | | | | | | | Gems like rspec-rails depend on `ARGV` being shifted, and `scaffold` (for example) not being the first item in `ARGV`. This should allow rspec-rails to be passing on Rails master.
| | * Remove unused method `namespaced_file_path`Fumiaki MATSUSHIMA2016-12-141-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | ``` $ git grep namespaced_file_path railties/lib/rails/generators/named_base.rb: def namespaced_file_path railties/lib/rails/generators/named_base.rb: @namespaced_file_path ||= namespaced_class_path.join("/") ```
| | * use appropriate type for `rc` optionyuuji.yaginuma2016-12-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | This fixes the following warning. ``` Expected boolean default value for '--rc'; got "~/.railsrc" (string) ```
| | * Merge pull request #27042 from kirs/yaml-schema-cacheRafael França2016-12-131-2/+2
| | |\ | | | | | | | | Schema cache in YAML
| | | * Use YAML to serialize schema cacheKir Shatrov2016-11-271-2/+2
| | | |
| | * | Don't assign a default to `webpack`.Kasper Timm Hansen2016-12-121-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Unintentionally makes `--webpack` implied on `rails new apper`. If passed `--webpack` Thor assigns `"webpack"` to `options[:webpack]`, so we can check for that instead of `"base"`.
| | * | Add option to trigger JS framework-specific installation as part of webpack ↵David Heinemeier Hansson2016-12-122-2/+17
| | | | | | | | | | | | | | | | setup
| | * | Oops!Akira Matsuda2016-12-121-1/+1
| | | | | | | | | | | | | | | | namespace_ladder can be nil here
| | * | :nail_care:Akira Matsuda2016-12-121-2/+2
| | | | | | | | | | | | | | | | Don't expect the caller of this method to know that the return value has an extra " "
| | * | run `webpack` command only when webpack option is specifiedyuuji.yaginuma2016-12-101-1/+1
| | | |
| | * | gemfile entry method need to return an empty array rather than nil (#27318)Yuji Yaginuma2016-12-091-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the following error when executing rails new command. ``` (erb):9:in `block in template': undefined method `comment' for nil:NilClass (NoMethodError) ``` Follow up to #27288
| | * | Use GitHub webpacker until closer to releaseDavid Heinemeier Hansson2016-12-091-1/+1
| | | |
| | * | Basic --webpack delegation to new webpacker gem (#27288)David Heinemeier Hansson2016-12-092-1/+16
| | | |
| | * | Show message if Yarn is not installed (#27312)Guillermo Iguaran2016-12-091-1/+8
| | | |
| | * | Only have Yarn bundling commented out as we cant be sure Yarn is installedDavid Heinemeier Hansson2016-12-082-5/+2
| | | |
| | * | Fix ability to run railties test suiteDavid Heinemeier Hansson2016-12-081-0/+1
| | | |
| | * | fix broken tests (#27308)Yuji Yaginuma2016-12-081-3/+2
| | | | | | | | | | | | Follow up to 0d20530e5edfd7d00fbc2a38ef5f87eca6ccc924
| | * | specify `skip_yarn` option in API-only Application (#27309)Yuji Yaginuma2016-12-082-3/+12
| | | |
| | * | Merge pull request #25051 from bf4/extract_notes_as_binarySean Griffin2016-12-081-1/+1
| | |\ \ | | | | | | | | | | Extract notes as binary
| | | * | Extract notes from files in binaryBenjamin Fleischer2016-09-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevents: ArgumentError: invalid byte sequence in UTF-8 railties/lib/rails/source_annotation_extractor.rb:115:in `=~' railties/lib/rails/source_annotation_extractor.rb:115:in `block in extract_annotations_from' And there's no reason we need to interpret the files as UTF-8 when scanning for annotations. Applies to Rails 4.2 as well.
generated by cgit v1.2.3 (git 2.25.1) at 2025-01-17 08:00:21 +0000