Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #23359 from kamipo/make_to_primary_key | Rafael França | 2016-02-01 | 4 | -40/+5 |
|\ | | | | | Make to primary key instead of an unique index for internal tables | ||||
| * | Make to primary key instead of an unique index for internal tables | Ryuta Kamizono | 2016-01-31 | 4 | -40/+5 |
| | | |||||
* | | Remove development dependencies from actioncable.gemspec | Matthew Draper | 2016-02-01 | 1 | -9/+0 |
| | | | | | | | | None of the other components use them, so we should be consistent. | ||||
* | | Merge pull request #23373 from ↵ | Rafael França | 2016-02-01 | 3 | -17/+1 |
|\ \ | | | | | | | | | | | | | kamipo/remove_duplicated_composite_primary_key_tests Remove duplicated composite primary key tests | ||||
| * | | Remove duplicated composite primary key tests | Ryuta Kamizono | 2016-02-01 | 3 | -17/+1 |
| | | | |||||
* | | | Merge pull request #23389 from kamipo/extract_explain_pretty_printer | Rafael França | 2016-02-01 | 7 | -119/+137 |
|\ \ \ | | | | | | | | | Extract `ExplainPrettyPrinter` to appropriate files | ||||
| * | | | Extract `ExplainPrettyPrinter` to appropriate files | Ryuta Kamizono | 2016-02-01 | 7 | -119/+137 |
| |/ / | |||||
* / / | Revert "Merge pull request #23366 from maclover7/add-configuation-ar-docs" | Rafael Mendonça França | 2016-02-01 | 1 | -102/+0 |
|/ / | | | | | | | | | | | | | This reverts commit 96355e87cba247246234386b0af9273cc5d59db9, reversing changes made to a00c36feea6c0271b5ad48a949ef294514fdef52. See https://github.com/rails/rails/pull/23366#issuecomment-177714429 | ||||
* | | Merge pull request #23372 from kamipo/use_index_in_create_in_test_schema | Rafael França | 2016-02-01 | 2 | -11/+9 |
|\ \ | | | | | | | Use `t.index` in `create_table` instead of `add_index` in test schema | ||||
| * | | Use `t.index` in `create_table` instead of `add_index` in test schema | Ryuta Kamizono | 2016-01-31 | 2 | -11/+9 |
| | | | | | | | | | | | | For reduce bootstrap queries in tests. | ||||
* | | | Merge pull request #23374 from prathamesh-sonpatki/migration-compatiblity-tests | Rafael França | 2016-01-31 | 1 | -0/+30 |
|\ \ \ | | | | | | | | | Added test for backward compatibility of null constraints on timestamp columns | ||||
| * | | | Added test for backward compatibility of null constraints on timestamp columns | Prathamesh Sonpatki | 2016-01-31 | 1 | -0/+30 |
| |/ / | |||||
* | | | :speak_no_evil: :warning: | Akira Matsuda | 2016-02-01 | 2 | -2/+6 |
| | | | |||||
* | | | Merge pull request #23360 from kamipo/oid_money_precision_is_unused | Rafael França | 2016-01-31 | 2 | -8/+0 |
|\ \ \ | | | | | | | | | `OID::Money.precision` is unused since #15239 | ||||
| * | | | `OID::Money.precision` is unused since #15239 | Ryuta Kamizono | 2016-01-31 | 2 | -8/+0 |
| | |/ | |/| | | | | | | | | | | | | | | | | | | | p PostgreSQLAdapter::OID::Money.precision # => 19 p PostgreSQLAdapter::OID::Money.new.precision # => nil | ||||
* | | | Merge pull request #23388 from prathamesh-sonpatki/update-gemfile-lock | Rafael França | 2016-01-31 | 1 | -1/+0 |
|\ \ \ | | | | | | | | | Update Gemfile.lock | ||||
| * | | | Update Gemfile.lock | Prathamesh Sonpatki | 2016-02-01 | 1 | -1/+0 |
|/ / / | | | | | | | | | | - Leftover from https://github.com/rails/rails/commit/93abf58787396661230f31c7a2f58c18f30dbec9. | ||||
* | | | Merge pull request #23380 from ↵ | Arthur Nogueira Neves | 2016-01-31 | 1 | -8/+13 |
|\ \ \ | | | | | | | | | | | | | | | | | yui-knk/override_calculate_constructables_at_each_concrete_class Remove `case macro` from `calculate_constructable` | ||||
| * | | | Remove `case macro` from `calculate_constructable` | yui-knk | 2016-02-01 | 1 | -8/+13 |
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | Rails has abstract Reflection classes (`MacroReflection`, `AssociationReflection` etc.) and concrete Reflection classes (e.g. `HasManyReflection`, `HasOneReflection` etc.). In many case `calculate_constructable` returns `true`, so change `calculate_constructable` to always return `true` and override this method if necessary. | ||||
* | | | Merge pull request #23387 from mcfiredrill/clarify-touching-callbacks | Kasper Timm Hansen | 2016-01-31 | 1 | -0/+2 |
|\ \ \ | | | | | | | | | clarify the touch true option does not trigger after_save/update [ci skip] | ||||
| * | | | clarify the touch true option does not trigger after_save/update [ci skip] | Tony Miller | 2016-02-01 | 1 | -0/+2 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I've gotten tripped up more than a few times on this, thinking that using `belongs_to` with `touch: true` would trigger my after_save or after_update callbacks. The same text is in the documentation for the touch method itself, but I think its helpful to repeat it again here. It might save people some time. | ||||
* | | | | Merge pull request #23370 from maclover7/actioncable-main-travis-build | Guillermo Iguaran | 2016-01-31 | 2 | -10/+0 |
|\ \ \ \ | | | | | | | | | | | Move Action Cable back to the main build | ||||
| * | | | | Remove unused method | Jon Moss | 2016-01-31 | 1 | -9/+0 |
| | | | | | |||||
| * | | | | Move Action Cable back to the main build | Jon Moss | 2016-01-30 | 1 | -1/+0 |
| | |_|/ | |/| | | |||||
* | | | | Merge pull request #22857 from bdewater/interlock-doc | Vipul A M | 2016-02-01 | 4 | -8/+9 |
|\ \ \ \ | | | | | | | | | | | Update middleware docs regarding ActionDispatch::LoadInterlock [ci skip] | ||||
| * | | | | Update middleware docs regarding ActionDispatch::LoadInterlock [ci skip] | Bart de Water | 2016-01-31 | 4 | -9/+10 |
| | | | | | |||||
* | | | | | Merge pull request #23169 from y-yagi/rake_proxy_in_engine | Kasper Timm Hansen | 2016-01-31 | 6 | -40/+127 |
|\ \ \ \ \ | | | | | | | | | | | | | make rake proxy work in rails engines | ||||
| * | | | | | make rake proxy work in rails engines | yuuji.yaginuma | 2016-01-31 | 6 | -40/+127 |
| | | | | | | |||||
* | | | | | | Merge pull request #23379 from ↵ | Kasper Timm Hansen | 2016-01-31 | 1 | -22/+25 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | yui-knk/define_association_class_on_each_concrete_class Each concrete classes have responsibility to return `association_class` | ||||
| * | | | | | | Each concrete classes have responsibility to return `association_class` | yui-knk | 2016-01-31 | 1 | -22/+25 |
| | | | | | | | |||||
* | | | | | | | Drop the runtime dependency on coffee-rails | Matthew Draper | 2016-02-01 | 1 | -1/+0 |
| | | | | | | | |||||
* | | | | | | | Wrangle the asset build into something that sounds more general | Matthew Draper | 2016-02-01 | 18 | -17/+51 |
| | | | | | | | |||||
* | | | | | | | Merge pull request #23369 from maclover7/actioncable-assets-redux | Matthew Draper | 2016-02-01 | 11 | -7/+49 |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Action Cable Assets Compilation redux | ||||
| * | | | | | | | Add Action Cable asset building as release step | Jon Moss | 2016-01-30 | 1 | -0/+1 |
| | | | | | | | | |||||
| * | | | | | | | Improvements and reorganization of assets | Jon Moss | 2016-01-30 | 10 | -3/+10 |
| | | | | | | | | |||||
| * | | | | | | | Add task to create precompiled action_cable.js and reorganize to accommodate | Javan Makhmali | 2016-01-30 | 10 | -7/+41 |
| | |_|_|/ / / | |/| | | | | | |||||
* | | | | | | | Merge pull request #23368 from ↵ | Santiago Pastorino | 2016-01-31 | 1 | -3/+0 |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | y-yagi/remove_backward_compatibility_code_for_minitest_4 remove backward compatibility code for Minitest 4 | ||||
| * | | | | | | | remove backward compatibility code for Minitest 4 | yuuji.yaginuma | 2016-01-30 | 1 | -3/+0 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The master branch is required Ruby 2.2.2+, for the Ruby 2.2 is bundled Minitest 5.4.3, I think backward compatibility code for Minitest 4 is unnecessary. | ||||
* | | | | | | | | Wait for EventMachine to finish starting | Matthew Draper | 2016-02-01 | 1 | -0/+1 |
| |_|/ / / / / |/| | | | | | | |||||
* | | | | | | | Merge pull request #23371 from kamipo/remove_odd_backtick | Vipul A M | 2016-01-31 | 1 | -1/+1 |
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | Remove odd ` [ci skip] | ||||
| * | | | | | | Remove odd ` [ci skip] | Ryuta Kamizono | 2016-01-31 | 1 | -1/+1 |
|/ / / / / / | |||||
* | | | | | | Merge pull request #23366 from maclover7/add-configuation-ar-docs | Jon Moss | 2016-01-30 | 1 | -0/+102 |
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | Add configuration section to "Active Record Basics" guide | ||||
| * | | | | | Add configuration section to "Active Record Basics" guide | Jon Moss | 2016-01-30 | 1 | -0/+102 |
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is to fill in some missing information as apart of #22931. It's on purpose that the sample `Message` model inherits from `ActiveRecord::Base` -- Active Record is not meant to be coupled to Rails, and we can't guarantee that users outside of the Rails world will have an `ApplicationRecord` class that inherits from `ActiveRecord::Base`. [ci skip] | ||||
* | | | | | Merge pull request #23364 from maclover7/fix-23358 | Jon Moss | 2016-01-30 | 1 | -1/+1 |
|\ \ \ \ \ | | | | | | | | | | | | | Small cleanup to Testing Guide | ||||
| * | | | | | Small cleanup to Testing Guide | Jon Moss | 2016-01-30 | 1 | -1/+1 |
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - inbuilt --> built-in - Remove random spaces from code examples [ci skip] | ||||
* | | | | | Merge pull request #23358 from vipulnsward/testing-guide-pass-3 | Vipul A M | 2016-01-31 | 1 | -19/+28 |
|\ \ \ \ \ | | | | | | | | | | | | | Pass 3 over testing guide | ||||
| * | | | | | Pass 3 over testing guide | Vipul A M | 2016-01-31 | 1 | -19/+28 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Various grammar fixes - Added assertions for update controller action tests - Added user helper tests - Fix typos [ci skip] | ||||
* | | | | | | Merge pull request #23363 from pra85/patch-1 | Vipul A M | 2016-01-31 | 1 | -1/+1 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | typo fix [ci skip] | ||||
| * | | | | | | typo fix [ci skip] | Prayag Verma | 2016-01-31 | 1 | -1/+1 |
|/ / / / / / | | | | | | | | | | | | | Spelling mistake - direcotry > directory | ||||
* | | | | | | Merge pull request #23362 from vipulnsward/20821-save-doc-update | Jon Moss | 2016-01-30 | 1 | -3/+3 |
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | - Updated persistence documentation to make it clear that save and sa… |