- 15 7月, 2018 1 次提交
-
-
由 Jeremy Baker 提交于
-
- 14 7月, 2018 3 次提交
-
-
由 George Claghorn 提交于
-
由 George Claghorn 提交于
-
由 George Claghorn 提交于
-
- 13 7月, 2018 5 次提交
-
-
由 George Claghorn 提交于
-
由 Rafael França 提交于
[ci skip] `sanitizer_vendor` will be removed in Rails 6
-
由 Rafael França 提交于
e4e1b620 broke `to_param` handling:
-
由 Rafael França 提交于
Enable `Layout/EmptyLinesAroundBlockBody` to reduce review cost in the future
-
由 Edouard CHIN 提交于
- There was an issue inside controller tests where order params were not respected, the reason was because we were calling `Hash#to_query` which sorts the results lexicographically. 1e4e1b62 fixed that issue by not using `to_query` but instead a utility function provided by rack. - However with the fix came another issue where it's now no longer possible to do this ``` post :foo, params: { user: User.first } # Prior to the patch the controller will receive { "user" => "1" } # Whereas now you get { "user": "#<User: ...>" } ``` The fix in this PR is to modify `Hash#to_query` to sort only when it doesn't contain an array structure that looks something like "bar[]" Ref https://github.com/rails/rails/pull/33341#issuecomment-404039396
-
- 12 7月, 2018 4 次提交
-
-
由 Ryuta Kamizono 提交于
We sometimes ask "
✂ ️ extra blank lines" to a contributor in reviews like https://github.com/rails/rails/pull/33337#discussion_r201509738. It is preferable to deal automatically without depending on manpower. -
由 George Claghorn 提交于
-
由 George Claghorn 提交于
We've never intentionally tripped this retry in production. Add retries for specific errors as needed.
-
由 Rafael França 提交于
Add missing env & config dependency to `rake db:seed`
-
- 11 7月, 2018 3 次提交
-
-
-
由 Rafael França 提交于
Fix stubbed methods in test cases
-
由 bogdanvlviv 提交于
Remove returning of `false` value for stubbed `lock_thread=` methods since there aren't any needs in it. Remove unnecessary returning of `true` for stubbed `drop_database` method. Follow up #33309. Related to #33162, #33326.
-
- 10 7月, 2018 7 次提交
-
-
由 Ryuta Kamizono 提交于
Replace shallow mocks with Ruby classes
-
由 Tobias Bielohlawek 提交于
-
由 utilum 提交于
While preparing this I realised that some stubbed returns values serve no purpose, so this patch drops those as well. Step 3 in #33162
-
由 Ryuta Kamizono 提交于
-
由 Rafael França 提交于
Reduce mocking by testing value instead of method call
-
由 utilum 提交于
Step 2 in #33162
-
由 Rafael França 提交于
Remove unnecessary Mocha stubs
-
- 09 7月, 2018 7 次提交
-
-
由 Rafael França 提交于
Add rake notes and SOURCE_ANNOTATION_DIRECTORIES deprecation to Changelog
-
由 Annie-Claude Côté 提交于
* SOURCE_ANNOTATION_DIRECTORIES deprecation * Deprecation of `rake notes`, use `rails notes` instead * Deprecation of `rails notes:custom ANNOTATION=custom`, `rails notes:optimize`, `rails notes:todo`, and `rails notes:fixme` in favor of passing `-annotations` or `-a` to `rails notes` * They have all been deprecrated in https://github.com/rails/rails/pull/33220
-
由 Ryuta Kamizono 提交于
Related #31201. If creating custom primary key (like a string) in SQLite, it would also create an internal index implicitly which named begin with "sqlite_". It need to be hidden since the internal object names are reserved and prohibited for public use. See https://www.sqlite.org/fileformat2.html#intschema Fixes #33320.
-
由 Ryuta Kamizono 提交于
-
由 Ryuta Kamizono 提交于
Fix default value for mysql time types with specified precision
-
由 Kasper Timm Hansen 提交于
rails server: Allow to explicitly specify whether to output Rails's log to stdout
-
由 Kasper Timm Hansen 提交于
Fix rubocop offense introduced in 161ed37d
-
- 08 7月, 2018 10 次提交
-
-
由 bogdanvlviv 提交于
We prefer double quotes over single quotes. Fixes: ``` railties/lib/rails/commands/server/server_command.rb:279:39: C: Style/StringLiterals: Prefer double-quoted strings unless you need single quotes to avoi d extra backslashes for escaping. original_options.concat [ '-u', using ] ``` Related to 161ed37d
-
由 George Claghorn 提交于
-
由 Markus Doits 提交于
Before Rails' logger output is mirrored to std out if: * environment is development and * the process is not daemonized It was not possible to change that behaviour, e.g. to disable log output in that case or enable it in other cases. Now you can explicitly disable or enable output with the new command line switch `--log-to-stdout`, regardless of any other circumstances. ``` // enable output in production rails server -e production --log-to-stdout // disable output in development rails server -e development --no-log-to-stdout ``` Enabling output when daemonized still makes no sense (since tty is detached), but this is ignored for now. If the command line flag is not specified, old behaviour still applies, so this change is completely backward compatible.
-
由 Kasper Timm Hansen 提交于
Documentation and some tests for `assert_no_difference`
-
由 lxxxvi 提交于
-
由 Kasper Timm Hansen 提交于
Merge pull request #33317 from bogdanvlviv/has_secure_password-use-recovery_password-instead-of-activation_token has_secure_password: use `recovery_password` instead of `activation_token`
-
由 bogdanvlviv 提交于
Since we have `has_secure_token`, it is too confusing to use `_token` suffix with `has_secure_password`. Context https://github.com/rails/rails/pull/33307#discussion_r200807185
-
由 George Claghorn 提交于
-
由 Ryuta Kamizono 提交于
Improve docs/test of `has_secure_password`
-
由 George Claghorn 提交于
Consider the following model definitions: class User < ApplicationRecord has_one_attached :avatar end class Group < ApplicationRecord has_one_attached :avatar end If you attempt to reflect on the User model's avatar attachment via User.reflect_on_attachment, you could receive a reflection for the Group model's avatar attachment. Fix this by ensuring that each model class uses its own Hash object to track attachment reflections.
-