aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* fix new warning in ruby 2.4yuuji.yaginuma2016-12-142-2/+2
| | | | | | | | | | | | | | This fixes the following warning. ``` test/caching_test.rb:986: warning: parentheses after method name is interpreted as test/caching_test.rb:986: warning: an argument list, not a decomposed argument test/cases/adapters/mysql2/reserved_word_test.rb:146: warning: parentheses after method name is interpreted as test/cases/adapters/mysql2/reserved_word_test.rb:146: warning: an argument list, not a decomposed argument ``` Ref: https://github.com/ruby/ruby/commit/65e27c8b138d6959608658ffce2fa761842b8d24
* form_with/fields: Don't output ids by defaultKasper Timm Hansen2016-12-134-206/+243
| | | | | | | | | | | Continuing 67f81cc where we decided not to output ids by default in the new form helpers. Went with @dhh's suggestion of just requiring ids on fields being labelled: https://github.com/rails/rails/issues/25197#issuecomment-231797117 Seems okay enough.
* Add CHANGELOG entry to #27042Rafael Mendonça França2016-12-131-0/+4
| | | | [ci skip]
* Merge pull request #27042 from kirs/yaml-schema-cacheRafael França2016-12-137-17/+435
|\ | | | | Schema cache in YAML
| * Use YAML to serialize schema cacheKir Shatrov2016-11-277-17/+435
| |
* | Merge pull request #27293 from y-yagi/fix_duplicable_with_2_4Kasper Timm Hansen2016-12-133-35/+71
|\ \ | | | | | | change return value of `duplicable?` with Ruby 2.4+
| * | change return value of `duplicable?` with Ruby 2.4+yuuji.yaginuma2016-12-133-35/+71
|/ / | | | | | | | | | | | | `NilClass`, `FalseClass`, `TrueClass`, `Symbol` and `Numeric` can dup with Ruby 2.4+. Ref: https://bugs.ruby-lang.org/issues/12979
* | 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
* | :nail_care:Akira Matsuda2016-12-131-2/+1
| |
* | nil, true, 1, etc. don't raise on #dup since Ruby 2.4Akira Matsuda2016-12-131-1/+1
| | | | | | | | https://bugs.ruby-lang.org/issues/12979
* | Keep AS::XmlMini::PARSING["decimal"].call('') returning 0Akira Matsuda2016-12-131-1/+11
| | | | | | | | | | BigDecimal('an invalid string') has changed its behavior to raise an ArgumentError since 1.3.0 https://bugs.ruby-lang.org/issues/10286
* | Missing require "yaml"Akira Matsuda2016-12-121-0/+1
| |
* | 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 " "
* | Merge pull request #27328 from kamipo/add_changelog_for_26687Eileen M. Uchitelle2016-12-111-0/+7
|\ \ | | | | | | Add a changelog entry for #26687 [ci skip]
| * | Add a changelog entry for #26687 [ci skip]Ryuta Kamizono2016-12-121-0/+7
|/ /
* | Merge pull request #27330 from kamipo/remove_duplicated_lineGuillermo Iguaran2016-12-111-1/+0
|\ \ | | | | | | Remove duplicated line
| * | Remove duplicated lineRyuta Kamizono2016-12-111-1/+0
|/ /
* | Merge pull request #27323 from kamipo/add_changelog_for_25227Matthew Draper2016-12-111-0/+4
|\ \ | | | | | | Add a changelog entry for #25227 [ci skip]
| * | Add a changelog entry for #25227 [ci skip]Ryuta Kamizono2016-12-101-0/+4
| | |
* | | Merge pull request #27322 from kamipo/fix_ci_failureMatthew Draper2016-12-112-2/+2
|\ \ \ | |/ / |/| | Fix CI failure caused by #25227 and #25280 were merged at the same time
| * | Fix CI failure caused by #25227 and #25280 were merged at the same timeRyuta Kamizono2016-12-102-2/+2
|/ /
* | Merge pull request #25280 from ↵Sean Griffin2016-12-102-0/+16
|\ \ | | | | | | | | | | | | kamipo/prevent_range_error_for_belongs_to_associations Prevent `RangeError` for `belongs_to` associations
| * | Prevent `RangeError` for `belongs_to` associationsRyuta Kamizono2016-10-102-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | Currently to access `belongs_to` associations raises a `RangeError` if foreign key attribute has out of range value. It should return a nil value rather than raising a `RangeError`. Fixes #20140.
* | | Merge pull request #25227 from kamipo/numeric_value_out_of_rangeSean Griffin2016-12-107-8/+26
|\ \ \ | | | | | | | | Translate numeric value out of range to the specific exception
| * | | Translate numeric value out of range to the specific exceptionRyuta Kamizono2016-12-067-8/+26
| | | | | | | | | | | | | | | | Raise `ActiveRecord::RangeError` when values that executed are out of range.
* | | | Merge pull request #27320 from ↵Matthew Draper2016-12-101-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | y-yagi/run_webpack_command_only_when_webpack_option_is_specified run `webpack` command only when webpack option is specified
| * | | | 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
| | | |
* | | | Merge pull request #27314 from kamipo/add_changelog_for_25451Matthew Draper2016-12-101-8/+13
|\ \ \ \ | | | | | | | | | | Add a changelog entry for #25451 [ci skip]
| * | | | Add a changelog entry for #25451 [ci skip]Ryuta Kamizono2016-12-101-8/+13
| | | | |
* | | | | Do not raise exception when content_type is a empty stringRafael Mendonça França2016-12-092-1/+6
|/ / / / | | | | | | | | | | | | | | | | When content type header is blank we were raising an exception because `empty?` was being called on nil.
* | | | Merge pull request #27248 from stefanmb/masterSean Griffin2016-12-094-5/+67
|\ \ \ \ | | | | | | | | | | Idempotent option for after_commit :destroy callback
| * | | | Emulate db trigger behaviour for after_commit :destroy, :updateStefan Budeanu2016-12-094-5/+67
|/ / / / | | | | | | | | | | | | | | | | | | | | Race conditions can occur when an ActiveRecord is destroyed twice or destroyed and updated. The callbacks should only be triggered once, similar to a SQL database trigger.
* | | | Show message if Yarn is not installed (#27312)Guillermo Iguaran2016-12-091-1/+8
| | | |
* | | | Merge pull request #27311 from y-yagi/remove_assert_sendKasper Timm Hansen2016-12-091-1/+0
|\ \ \ \ | | | | | | | | | | remove deprecated `assert_send` from assertion list [ci skip]
| * | | | remove deprecated `assert_send` from assertion list [ci skip]yuuji.yaginuma2016-12-091-1/+0
|/ / / / | | | | | | | | | | | | | | | | `assert_send` was deprecated in minitest 5.10.0. Ref: https://github.com/seattlerb/minitest/commit/f95ef007ec64d956d88fedc37fd2ed7e106b777e
* | | | 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 #27303 from toshimaru/fix/partial-variable-blockMatthew Draper2016-12-094-5/+17
|\ \ \ \ | | | | | | | | | | Enable `block` variable in view
| * | | | split DELEGATION_RESERVED_METHOD_NAMES in halfToshimaru2016-12-095-10/+17
| | | | |
| * | | | Enable `block` variable in viewToshimaru2016-12-093-1/+6
| | | | | | | | | | | | | | | | | | | | Remove `block` keyword from reserved method names
* | | | | Resolve association class correctly when assigning ids on a through associationMatthew Draper2016-12-093-1/+12
| | | | |
* | | | | Make ActiveModel::Errors backward compatible with 4.2Rafael Mendonça França2016-12-082-2/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a Error object was serialized in the database as YAML in the Rails 4.2 version, if we load in the Rails 5.0 version it will miss the @details instance variable so methods like #clear and #add will start to fail.
* | | | | Merge pull request #27304 from utilum/fix_test_breakerSean Griffin2016-12-081-0/+1
|\ \ \ \ \ | |/ / / / |/| | | | Fix test breaking error uninitialized constant ActiveRecord::Type::Value
generated by cgit v1.2.3 (git 2.25.1) at 2025-01-17 02:21:06 +0000