- 07 4月, 2018 1 次提交
-
-
由 Edouard CHIN 提交于
-
- 06 4月, 2018 1 次提交
-
-
由 Edouard CHIN 提交于
If you have a regular test that have a teardown block, and for any reason an exception get raised, ActiveSupport will not run subsequent after_teardown method provided by other module or gems. One of them being the ActiveRecord::TestFixtures which won't rollback the transation when the test ends making all subsequent test to be in a weird state. The default implementation of minitest is to run all teardown methods from the user's test, rescue all exceptions, run all after_teardown methods provided by libraries and finally re-raise the exception that happened in the user's teardown method. Rails should do the same.
-
- 05 4月, 2018 12 次提交
-
-
由 Rafael França 提交于
Add ujs desc to rakefile in actionview
-
由 Yoshiyuki Hirano 提交于
* Seems the desc of Rake::TestTask.new is not displayed * Use comment instead of desc
-
由 Yoshiyuki Hirano 提交于
-
由 Rafael França 提交于
Add custom RuboCop for `assert_not` over `refute`
-
由 Rafael Mendonça França 提交于
Stop mutating body response
-
由 Rafael França 提交于
Add #dig to ActionDispatch::Request::Session
-
由 Rafael França 提交于
2.6 warnings: passing splat keyword arguments as a single Hash
-
由 Rafael França 提交于
[ci skip] Add :private option to delegation doc
-
由 Yoshiyuki Hirano 提交于
-
由 Rafael França 提交于
Optimize the code inside AR::QueryCache middleware
-
由 Rafael França 提交于
Remove superfluous `ActionController::`
-
由 Francois Chagnon 提交于
-
- 04 4月, 2018 15 次提交
-
-
由 Bogdan Gusiev 提交于
-
由 Eileen M. Uchitelle 提交于
Add missing dots at the end of comments in environment file templates
-
由 Eileen M. Uchitelle 提交于
[ci skip] Add :private option to delegate section in guide
-
由 Yoshiyuki Hirano 提交于
-
由 utilum 提交于
Ruby 2.6.0 warns about this.
-
由 claudiob 提交于
### Summary The `session` object is not a real Hash but responds to many methods of Hash such as `[]`, `[]`, `fetch`, `has_key?`. Since Ruby 2.3, Hash also supports a `dig` method. This commit adds a `dig` method to `ActionDispatch::Request::Session` with the same behavior as `Hash#dig`. This is useful if you store a hash in your session, such as: ```ruby session[:user] = { id: 1, avatar_url: "http://example.org/nyancat.jpg" } ``` Then you can shorten your code from `session[:user][:avatar_url]` to `session.dig :user, :avatar_url`. ### Other Information I cherry-picked a commit from https://github.com/rails/rails/pull/23864, and modify a bit. The changes are below: * Converts only the first key to a string adjust to the `fetch` method. * Fixes a test case because we cannot use the indifferent access since ee5b621e.
-
由 Geoff Lee 提交于
If @app.call returns an object that is saved (for e.g., in a constant), the mutation results in a continuing cycle of wrapping the body in Rack::BodyProxy, eventually leading to SystemStackError ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ On branch fix-return-response-mutation-rack-logger - Tue 3 Apr 2018 19:54:28 PDT by Geoff Lee <geoff.lee@lendesk.com>
-
由 Daniel Colson 提交于
73e7aab behaved as expected on codeship, failing the build with exactly these RuboCop violations. Hopefully `rubocop -a` will have been enough to get a passing build!
-
由 Daniel Colson 提交于
Since at least cf4afc4d we have preferred `assert_not` methods over `refute` methods. I have seen plenty of comments in PRs about this, and we have tried to fix it a few times (5294ad82, e45f176b, 8910f12f, 41f50beb, d4cfd543, 48a183ec, and 211adb47), but the `refute` methods keep sneaking back in. This custom RuboCop will take care of enforcing this preference, so we don't have to think about it again. I suspect there are other similar stylistic preferences that could be solved with some custom RuboCops, so I will definitely keep my eyes open. `assert_not` over `assert !` might be a good candidate, for example. I wasn't totally sure if `ci/custom_cops` was the best place to put this, but nothing else seemed quite right. At one point I had it set up as a gem, but I think custom cops like this would have limited value in another context. I want to see how code climate handles the new cops before autocorrecting the existing violations. If things go as expected, I will push another commit with those corrections.
-
由 Ryuta Kamizono 提交于
It is wrongly appeared as instance public methods in the doc. http://api.rubyonrails.org/v5.1.6/classes/ActionCable/Channel/Callbacks.html http://api.rubyonrails.org/v5.1.6/classes/ActiveRecord/Timestamp.html
-
由 Yuji Yaginuma 提交于
Fix RenderingTest in railtie
-
由 Andrew White 提交于
Use `:default` option in order to set default value of `finalize_compiled_template_methods`
-
由 Yoshiyuki Hirano 提交于
Test class name is not `RoutingTest` but `RenderingTest`
-
由 Xavier Noria 提交于
Note that tab_yellow.gif exists, that is the one being used.
-
由 Xavier Noria 提交于
The credits page is gone after #32429, these images are now orphan.
-
- 03 4月, 2018 11 次提交
-
-
由 bogdanvlviv 提交于
Add dots in order to keep consistency between other comments in these files.
-
由 bogdanvlviv 提交于
Since we introduced default option for `class_attribute` and `mattr_accessor` family of methods and changed all occurrences of setting default values by using of `:default` option I think it would be fine to use `:default` option in order to set default value of `finalize_compiled_template_methods` since it expresses itself very well. Related to #29294, #32418
-
由 Andrew White 提交于
Cleanup credits related code as it is removed now in PR #32429
-
由 Prathamesh Sonpatki 提交于
-
由 Kasper Timm Hansen 提交于
Remove credits.html from Rails Guides
-
由 Ryuta Kamizono 提交于
Display db:drop description in ActiveRecord
-
由 Yoshiyuki Hirano 提交于
-
由 Ryuta Kamizono 提交于
Small doc fixes [ci skip]
-
由 Ryuta Kamizono 提交于
Doc fix added missing quote [ci skip]
-
由 Yoshiyuki Hirano 提交于
* refs #32420
-
由 Michael H 提交于
-