Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Add process.action_mailer notification to Instrumentation guide [ci skip] | Yoshiyuki Hirano | 2017-10-13 | 1 | -0/+16 |
| | |||||
* | Clarify comment [ci skip] | George Claghorn | 2017-10-13 | 1 | -1/+1 |
| | |||||
* | Merge pull request #30874 from koic/bump_rubocop_to_0_50_0 | Ryuta Kamizono | 2017-10-13 | 2 | -4/+4 |
|\ | | | | | Bump RuboCop 0.50.0 | ||||
| * | Bump RuboCop 0.50.0 | Koichi ITO | 2017-10-13 | 2 | -4/+4 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RuboCop 0.50.0 was released. https://github.com/bbatsov/rubocop/releases/tag/v0.50.0 And `rubocop-0-50` channel is available in Code Climate. https://github.com/codeclimate/codeclimate-rubocop/issues/107#issuecomment-336234260 This commit will bump RuboCop to 0.50.0. There are no new offences in this change. ```console % bundle exec rubocop --version 0.50.0 % bundle exec rubocop Inspecting 2350 files (snip) 2350 files inspected, no offenses detected ``` | ||||
* | | Merge pull request #30750 from k2nr/fix-active-job | Ryuta Kamizono | 2017-10-13 | 3 | -3/+3 |
|\ \ | |/ |/| | Yield with an error instance instead of error class | ||||
| * | Test exception message to ensure an exception instance is yielded | Kazunori Kajihiro | 2017-10-13 | 2 | -2/+2 |
| | | |||||
| * | Yield with an error instance instead of error class | Kazunori Kajihiro | 2017-09-29 | 1 | -1/+1 |
| | | |||||
* | | Accept variation keys in #preview and #variant | George Claghorn | 2017-10-12 | 2 | -7/+12 |
| | | |||||
* | | Introduce ActiveStorage::Blob#representation | George Claghorn | 2017-10-12 | 4 | -0/+73 |
| | | |||||
* | | Merge pull request #30867 from aditya-kapoor/guide-select_all-fix | Ryuta Kamizono | 2017-10-12 | 1 | -2/+2 |
|\ \ | | | | | | | fix the description for the `select_all` [ci skip] | ||||
| * | | fix the description for the `select_all` [ci skip] | Aditya Kapoor | 2017-10-12 | 1 | -2/+2 |
| | | | |||||
* | | | Merge pull request #30864 from willnet/enable-link-in-rdoc | Ryuta Kamizono | 2017-10-12 | 1 | -12/+12 |
|\ \ \ | | | | | | | | | [ci skip]Enable link to ActionDispatch::Integration::Session#process in rdoc | ||||
| * | | | [ci skip]Enable link to ActionDispatch::Integration::Session#process in rdoc | willnet | 2017-10-12 | 1 | -12/+12 |
| |/ / | |||||
* | | | Merge pull request #30863 from y-yagi/fix_warning_unused_variable | Ryuta Kamizono | 2017-10-12 | 1 | -2/+1 |
|\ \ \ | |/ / |/| | | Fix "warning: assigned but unused variable - message" | ||||
| * | | Fix "warning: assigned but unused variable - message" | yuuji.yaginuma | 2017-10-12 | 1 | -2/+1 |
|/ / | | | | | | | Ruby 2.5 warns about this. Ref: https://travis-ci.org/rails/rails/jobs/286338999 | ||||
* | | Merge pull request #30855 from boone/fix_typos | Rafael França | 2017-10-10 | 4 | -6/+6 |
|\ \ | | | | | | | Fix some typos [ci skip] | ||||
| * | | Fix some typos. | Mike Boone | 2017-10-10 | 4 | -6/+6 |
|/ / | |||||
* | | Merge pull request #30850 from timsly/safe-redirect-back | Rafael França | 2017-10-10 | 2 | -7/+37 |
|\ \ | | | | | | | Add allow_other_host option to redirect_back method | ||||
| * | | Add allow_other_host option to redirect_back method | Tim Masliuchenko | 2017-10-10 | 2 | -7/+37 |
| | | | |||||
* | | | Fix documentation [ci skip] | Rafael Mendonça França | 2017-10-10 | 1 | -1/+1 |
| | | | |||||
* | | | Merge pull request #30847 from mikeycgto/signed-cookie-rotation-updates | Rafael França | 2017-10-10 | 2 | -2/+22 |
|\ \ \ | |/ / |/| | | Update security guide for signed cookie rotations | ||||
| * | | Update security guide for signed cookie rotations | Michael Coyne | 2017-10-09 | 2 | -2/+22 |
|/ / | | | | | | | | | The example was slightly incorrect. This commit also adds a test case for this example to cookies middleware unit tests. | ||||
* | | Merge pull request #30595 from bogdanvlviv/use-railties-rails-command | Matthew Draper | 2017-10-10 | 6 | -249/+207 |
|\ \ | | | | | | | Improve railties' tests | ||||
| * | | Improve RakeTest#test_db_test_prepare_when_using_sql_format | bogdanvlviv | 2017-10-08 | 1 | -3/+1 |
| | | | | | | | | | | | | | | | - Remove redundant setting `RAILS_ENV` for `db:test:prepare`. `db:test:prepare` doesn't require it. | ||||
| * | | Invoke rails command inside the railties' test app with ↵ | bogdanvlviv | 2017-10-08 | 3 | -16/+12 |
| | | | | | | | | | | | | | | | | | | TestHelpers::Generation#rails See #30520 | ||||
| * | | Remove redundant execution of `Dir.chdir(app_path) { }` in railties' tests | bogdanvlviv | 2017-10-08 | 4 | -230/+194 |
| | | | |||||
* | | | Merge pull request #30836 from ↵ | Matthew Draper | 2017-10-10 | 2 | -2/+9 |
|\ \ \ | | | | | | | | | | | | | | | | | shioyama/generated_attribute_methods_include_mutex Include Mutex_m into GeneratedAttributeMethods instead of extending instance | ||||
| * | | | Add test for class of GeneratedAttributeMethods instance in ancestors | Chris Salzberg | 2017-10-09 | 1 | -0/+5 |
| | | | | |||||
| * | | | Include Mutex_m into module class instead of extending instance | Chris Salzberg | 2017-10-08 | 1 | -2/+4 |
| | | | | |||||
* | | | | Merge pull request #30832 from y-yagi/simplify_console_test | Ryuta Kamizono | 2017-10-10 | 1 | -16/+3 |
|\ \ \ \ | | | | | | | | | | | Simplify parse arguments in `ConsoleTest` | ||||
| * | | | | Simplify parse arguments in `ConsoleTest` | yuuji.yaginuma | 2017-10-08 | 1 | -16/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | If need a parse result of arguments, can obtain it by creating an instance of the command. | ||||
* | | | | | Merge pull request #30798 from ↵ | Sean Griffin | 2017-10-09 | 1 | -1/+1 |
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | bogdanvlviv/express-change_column_comment-as-public-api Express #change_column_comment as public api | ||||
| * | | | | | Express #change_column_comment as public api | bogdanvlviv | 2017-10-04 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implemented by #22911 Related to #30677 | ||||
* | | | | | | Merge pull request #30843 from y-yagi/use_released_gem | Ryuta Kamizono | 2017-10-10 | 2 | -12/+7 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Use released `sidekiq` instead of master version | ||||
| * | | | | | | Use released `sidekiq` instead of master version | yuuji.yaginuma | 2017-10-09 | 2 | -12/+7 |
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | The sidekiq 5.0.5 includes redis-rb 4.0 support. Ref: https://github.com/mperham/sidekiq/blob/90db3b84208cbb73a50d1a77a1dea97d3490ce70/Changes.md#505 | ||||
* | | | | | | Joined tables in association scope doesn't use the same aliases with the ↵ | Ryuta Kamizono | 2017-10-09 | 7 | -25/+36 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | parent relation's aliases Building association scope in join dependency should respect the parent relation's aliases to avoid using the same alias name more than once. Fixes #30681. | ||||
* | | | | | | All test cases for `exists?` places in `finder_test.rb` to ease to find the ↵ | Ryuta Kamizono | 2017-10-09 | 2 | -26/+26 |
| | | | | | | | | | | | | | | | | | | | | | | | | test cases | ||||
* | | | | | | Merge pull request #30839 from yhirano55/use_form_with_in_engine_guide | Ryuta Kamizono | 2017-10-09 | 2 | -9/+9 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Use `form_with` instead of `form_for` in engine guide [ci skip] | ||||
| * | | | | | | Use `form_with` instead of `form_for` in engine guide [ci skip] | Yoshiyuki Hirano | 2017-10-09 | 2 | -9/+9 |
| | |/ / / / | |/| | | | | |||||
* | | | | | | Remove meaningless named `construct_relation_for_association_calculations` | Ryuta Kamizono | 2017-10-09 | 2 | -6/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I don't think this is a good abstraction because the internal method is used only if the relation need to be applied join dependency. | ||||
* | | | | | | Merge pull request #30838 from ↵ | Ryuta Kamizono | 2017-10-09 | 1 | -19/+21 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | yhirano55/fix_rails_routes_log_in_getting_started_guide Fix the result of `rails routes` in Guide [ci skip] | ||||
| * | | | | | | Fix the result of `rails routes` in Guide [ci skip] | Yoshiyuki Hirano | 2017-10-09 | 1 | -19/+21 |
| |/ / / / / | |||||
* | | | | | | Fix `relation.exists?` with has_many through associations | Ryuta Kamizono | 2017-10-09 | 2 | -4/+12 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `relation.exists?` should reference correct aliases while joining tables of has_many through associations. | ||||
* | | | | | | Missing revision 5efd21bb7ee4b326d9c0d26fccad10fc59dd51f0 in redis-rb-4.0 branch | Ryuta Kamizono | 2017-10-09 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | Seems the branch has rebased. cc @jeremy | ||||
* | | | | | | Merge pull request #30837 from y-yagi/fix_typo_in_loading_error | Ryuta Kamizono | 2017-10-09 | 1 | -1/+1 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | Fix typo in loading error message | ||||
| * | | | | | | Fix typo in loading error message | yuuji.yaginuma | 2017-10-09 | 1 | -1/+1 |
|/ / / / / / | | | | | | | | | | | | | | | | | | | s/Action Record/Active Record/ | ||||
* | | | | | | Merge pull request #30748 from jeremy/redis-rb-4.0.1 | Jeremy Daer | 2017-10-08 | 9 | -30/+59 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | redis-rb 4.0 support | ||||
| * | | | | | | redis-rb 4.0 support | Jeremy Daer | 2017-10-08 | 9 | -30/+59 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Use `gem 'redis', '~> 4.0'` for new app Gemfiles * Loosen Action Cable redis-rb dep to `>= 3.3, < 5` * Bump redis-namespace for looser Redis version dep * Avoid using the underlying `redis.client` directly * Use `Redis.new` instead of `Redis.connect` | ||||
* | | | | | | | Merge pull request #30757 from jeremy/optional-dependency-hellscape | Jeremy Daer | 2017-10-08 | 3 | -7/+32 |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | Adapters bubble up gem version mismatches for their dependencies | ||||
| * | | | | | | | Distinguish missing adapter gems from load errors within the adapter | Jeremy Daer | 2017-10-08 | 3 | -7/+32 |
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * When the adapter is missing, raise an exception that points out config typos and missing Gemfile entries. (We can assume that a non-builtin adapter was used since these are always available.) * When loading an adapter raises a LoadError, prefix its error message to indicate that the adapter is likely missing an optional dependency. |