rails.git - Mirror of official rails repo with custom fixes.

	Commit message (Collapse)	Author	Age	Files	Lines
...
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #29720 from gaurish/ar_find_error_message_improvement	Rafael França	2017-08-11	4	-7/+11
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Return Not found Ids in ActiveRecord::NotFound
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Return Not found Ids in ActiveRecord::NotFound	Gaurish Sharma	2017-07-29	4	-7/+11
\| \| \|_\|_\|_\|_\|_\|_\|_\|/ / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This builds on top of 15e2da656f41af0124f7577858536f3b65462ad5. now it also returns exact Ids which were not found which will be debugging simple.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30193 from zverok/robust-postgres-duplicate-db	Rafael França	2017-08-11	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	More robust PostgreSQL database duplication check
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Update database duplication check	zverok	2017-08-11	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30201 from ↵	Rafael França	2017-08-11	1	-12/+8
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	kamipo/eval_active_storage_attached_name_only_once Evaluate `@active_storage_attached_#{name}` only once
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Evaluate `@active_storage_attached_#{name}` only once	Ryuta Kamizono	2017-08-12	1	-12/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Currently `"@active_storage_attached_#{name}"` in `define_method` is evaluated every call. It is better to evaluate it only once.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30202 from kamipo/add_null_false_to_active_storage_tables	Rafael França	2017-08-11	1	-10/+10
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add `null: false` to Active Storage tables
\| * \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add `null: false` to Active Storage tables	Ryuta Kamizono	2017-08-12	1	-10/+10
\| \|/ / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	These columns aren't intended nullable.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Specify `table.name` only when `scope.table` and `table` are different (#29058)	Ryuta Kamizono	2017-08-11	2	-4/+13
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fixes #29045.
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Remove unused `close_connection` in Action Cable tests (#30195)	Ryuta Kamizono	2017-08-11	2	-8/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Enable browser caching for blobs and variants (#30196)	Anton Khamets	2017-08-11	3	-0/+17
\|/ / / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Microsoft Azure --> Microsoft Azure Storage [ci skip]	Ryuta Kamizono	2017-08-12	1	-1/+1
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Refactor app_generator_test (#30192)	yhirano55	2017-08-11	1	-1/+8
\| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30186 from yhirano55/fix_generator_example_code	Robin Dupret	2017-08-11	1	-2/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix generator example code [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix example codes for generators guide [ci skip]	Yoshiyuki Hirano	2017-08-11	1	-2/+2
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30197 from ohbarye/fix-rdoc-code-formatting	Ryuta Kamizono	2017-08-11	3	-7/+7
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix RDoc formatting: `+` doesn't work with `@` [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix RDoc formatting: `+` doesn't work with `@`	ohbarye	2017-08-11	3	-7/+7
\|/ / / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	refs: https://github.com/rails/rails/pull/30161 ``` $ echo "+@size+" \| rdoc --pipe <p>+@size+</p> $ echo "<tt>@size</tt>" \| rdoc --pipe <p><code>@size</code></p> ``` [ci skip]
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30190 from ydakuka/patch-1	Vipul A M	2017-08-11	1	-2/+2
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	[ci skip] Update testing.md [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \| \|	Update testing.md	Yauheni Dakuka	2017-08-11	1	-2/+2
\|/ / / / / / / / / / / / / /
* \| \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30189 from koic/fix_formatting_of_active_storage_docs	Ryuta Kamizono	2017-08-11	7	-33/+33
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ / \|/\| \| \| \| \| \| \| \| \| \| \| \| \|	Fix formatting of Active Storage docs [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \| \|	Fix formatting of Active Storage docs [ci skip]	Koichi ITO	2017-08-11	7	-33/+33
\|/ / / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Follow up of #30188.
* \| \| \| \| \| \| \| \| \| \| \| \|	Merge pull request #30188 from y-yagi/fix_formatting_of_active_storage_docs	Ryuta Kamizono	2017-08-11	9	-36/+36
\|\ \ \ \ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|/ / / / / \|/\| \| \| \| \| \| \| \| \| \| \| \|	Fix formatting of Active Storage docs [ci skip]
\| * \| \| \| \| \| \| \| \| \| \| \|	Fix formatting of Active Storage docs [ci skip]	yuuji.yaginuma	2017-08-11	9	-36/+36
\|/ / / / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Use `+` instead of backquote. * Remove escape from class to be link
* \| \| \| \| \| \| \| \| \| \| /	Run Active Storage tests in CI (#30144)	George Claghorn	2017-08-10	6	-21/+21
\| \|_\|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #30183 from kamipo/reaper_run_after_conn_pool_initialized	Matthew Draper	2017-08-11	1	-2/+3
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Start `@reaper.run` after connection pool initialized
\| * \| \| \| \| \| \| \| \| \| \|	Start `@reaper.run` after connection pool initialized	Ryuta Kamizono	2017-08-11	1	-2/+3
\| \| \|_\|/ / / / / / / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Otherwise `ConnectionPool#reap` may run before `@connections` has initialized. https://travis-ci.org/rails/rails/jobs/263037427#L888-L890
* \| \| \| \| \| \| \| \| \| \|	Isolate ActiveStorage namespaces (#30095)	Dino Maric	2017-08-10	2	-1/+3
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	* Isolate ActiveStorage namespaces * Rename migrations task [Rafael Mendonça França + Dino Maric]
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #30164 from bogdanvlviv/improve-railties-generators-tests	Rafael França	2017-08-10	4	-45/+85
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Improve railties generators tests
\| * \| \| \| \| \| \| \| \| \| \|	`--skip-action-cable` pass throughs `rails plugin new`	bogdanvlviv	2017-08-10	3	-1/+11
\| \| \| \| \| \| \| \| \| \| \| \|
\| * \| \| \| \| \| \| \| \| \| \|	Improve plugin generator tests. Related to #30123.	bogdanvlviv	2017-08-10	2	-44/+74
\| \|/ / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Improve app generator tests. Ensure that generation `config/application.rb` is correct. Ensure that generation `config/application.rb` is correct.
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #30152 from bogdanvlviv/fix_random_ci_failure	Matthew Draper	2017-08-10	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix random CI failure due to non-deterministic sorting order
\| * \| \| \| \| \| \| \| \| \| \|	Fix random CI failure due to non-deterministic sorting order	bogdanvlviv	2017-08-09	1	-1/+1
\| \|/ / / / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	``` rails/activerecord$ bundle exec rake postgresql:test --verbose TESTOPTS="--seed=36062" Failure: AssociationsJoinModelTest#test_has_many_through_polymorphic_has_one [/home/travis/build/rails/rails/activerecord/test/cases/associations/join_model_test.rb:407]: --- expected +++ actual @@ -1 +1 @@ -[ #<Tagging id: 1, tag_id: 1, super_tag_id: 2, taggable_type: "Post", taggable_id: 1, comment: nil>, #<Tagging id: 2, tag_id: 1, super_tag_id: nil, taggable_type: "Post", taggable_id: 2, comment: nil> ] +#<ActiveRecord::Associations::CollectionProxy [ #<Tagging id: 2, tag_id: 1, super_tag_id: nil, taggable_type: "Post", taggable_id: 2, comment: nil>, #<Tagging id: 1, tag_id: 1, super_tag_id: 2, taggable_type: "Post", taggable_id: 1, comment: nil> ]> ```
* \| \| \| \| \| \| \| \| \| \|	Merge pull request #30174 from qsona/storage	Ryuta Kamizono	2017-08-10	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \ \ \| \|_\|/ / / / / / / / / \|/\| \| \| \| \| \| \| \| \| \|	Fix a typo in code comment [ci skip]
\| * \| \| \| \| \| \| \| \| \|	Fix a typo	qsona	2017-08-10	1	-1/+1
\|/ / / / / / / / / /
* \| \| \| \| \| \| \| \| \|	Revert "Revert "Set Ruby version in Gemfile and .ruby-version by default""	David Heinemeier Hansson	2017-08-09	4	-0/+24
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This reverts commit a8f59044fb18ce718367ffe4cd3ec0259c766b5f. See discussion on https://github.com/rails/rails/pull/30016.
* \| \| \| \| \| \| \| \| \|	Revert "Set Ruby version in Gemfile and .ruby-version by default"	schneems	2017-08-09	4	-24/+0
\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	This reverts commit fd7f978a5068921380308fe439af3923566c8f61.
* \| \| \| \| \| \| \| \| \|	Merge pull request #30170 from bogdanvlviv/fix_random_ci_failure_ar_dirty	Rafael França	2017-08-09	1	-4/+3
\|\ \ \ \ \ \ \ \ \ \ \| \|_\|_\|_\|_\|_\|_\|_\|_\|/ \|/\| \| \| \| \| \| \| \| \|	Fix random CI failure DirtyTest
\| * \| \| \| \| \| \| \| \|	Fix random CI failure DirtyTest:	bogdanvlviv	2017-08-09	1	-4/+3
\| \|/ / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	``` rails/activerecord$ bundle exec rake postgresql:test TESTOPTS="--seed=26536" ..(compressed)....F Failure: DirtyTest#test_save_should_not_save_serialized_attribute_with_partial_writes_if_not_present [/home/travis/build/rails/rails/activerecord/test/cases/dirty_test.rb:473]: Expected nil to not be nil. ```
* \| \| \| \| \| \| \| \|	Merge pull request #30161 from tjschuck/rdoc_code_format_fix	Robin Dupret	2017-08-09	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \| \|/ / / / / / / / \|/\| \| \| \| \| \| \| \|	Fix broken RDoc formatting
\| * \| \| \| \| \| \| \|	Fix broken RDoc formatting	T.J. Schuck	2017-08-09	1	-1/+1
\|/ / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	The `@` ivar format doesn’t work with RDoc's `+` code formatting; needs `<tt>`. [ci skip]
* \| \| \| \| \| \| \|	Merge pull request #30091 from yhirano55/fix_engine_command	Rafael França	2017-08-08	2	-0/+6
\|\ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix engine command
\| * \| \| \| \| \| \| \|	Fix engine command	Yoshiyuki Hirano	2017-08-06	2	-0/+6
\| \| \| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Merge pull request #30088 from y-yagi/use_the_same_key_as_presented_in_doc	Rafael França	2017-08-08	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Use the same key as presented in doc
\| * \| \| \| \| \| \| \| \|	Use the same key as presented in doc	yuuji.yaginuma	2017-08-09	1	-1/+1
\| \| \|_\|_\|_\|_\|/ / / \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Ref: https://github.com/rails/rails/blob/a8217bb594a625fb46a6af1488c0bde4aeac4986/railties/lib/rails/generators/rails/app/templates/config/storage.yml#L24
* \| \| \| \| \| \| \| \|	Merge pull request #30090 from yhirano55/fix_escaped_character	Rafael França	2017-08-08	1	-1/+1
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Fix non escaped character
\| * \| \| \| \| \| \| \| \|	Fix escaped character	Yoshiyuki Hirano	2017-08-06	1	-1/+1
\| \| \|/ / / / / / / \| \|/\| \| \| \| \| \| \|
* \| \| \| \| \| \| \| \|	Merge pull request #30108 from yui-knk/require_concurrent_map	Rafael França	2017-08-08	1	-0/+2
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Add missed `require`
\| * \| \| \| \| \| \| \| \|	Add missed `require`	yui-knk	2017-08-07	1	-0/+2
\| \| \|_\|_\|_\|_\|_\|_\|/ \| \|/\| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	`ActiveRecord::ConnectionAdapters::QueryCache::ConnectionPoolConfiguration` depends on `Concurrent::Map`.
* \| \| \| \| \| \| \| \|	Merge pull request #30112 from dixpac/filter_minitest_backtrace	Rafael França	2017-08-08	1	-0/+4
\|\ \ \ \ \ \ \ \ \ \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	ActiveStorage:Filter out Minitest backtrace.
\| * \| \| \| \| \| \| \| \|	Filter out Minitest backtrace.	dixpac	2017-08-07	1	-0/+4
\| \|/ / / / / / / / \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \| \|	Filter out Minitest backtrace while allowing backtrace from other libraries to be shown.