aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Undeprecate plural positional argumentJeremy Daer2016-09-212-14/+3
| | | | | | | | | | | | | | | | | | ```ruby pluralize people.count, 'person', 'people' ``` reads more naturally than ```ruby pluralize people.count, 'person', plural: 'people' ``` so let's not deprecate it. We could label both, but that's a mouthful: ```ruby pluralize people.count, singular: 'person', plural: 'people' ``` (The `plural:` kwarg shipped in 5.0.0, so we're keeping it.)
* Merge pull request #26562 from Neodelf/docsJon Moss2016-09-201-1/+1
|\ | | | | [ci skip] Replace to closing tag
| * [ci skip] Replace to closing tagAndrey Molchanov2016-09-201-1/+1
| |
* | Merge branch 'master' of github.com:rails/docrailsVijay Dev2016-09-205-8/+10
|\ \
| * | [ci skip] Fix wrong rdoc-ref links, the format is {}[] not []{}Prathamesh Sonpatki2016-09-141-2/+2
| | |
| * | [ci skip] Fix formatting of documentation of worker_pool method from ↵Prathamesh Sonpatki2016-09-071-1/+1
| | | | | | | | | | | | AC::Server::Base
| * | [ci skip] Fix formatting in Action Cable Connection::Base module docsPrathamesh Sonpatki2016-09-071-1/+1
| | |
| * | :nail_care: [ci skip]Prathamesh Sonpatki2016-09-071-1/+1
| | |
| * | Document that redirect* don't halt exucution. We explain more on this ↵Vipul A M2016-09-061-0/+2
| | | | | | | | | | | | behavior in 2.3.2, but dont specify that is not halting. Its better to not let developers be caught by surprise. [ci skip]
| * | [ci fix] Fix API documentation for Streams module from Action CablePrathamesh Sonpatki2016-09-061-4/+4
| | |
* | | Merge pull request #26558 from kamipo/restore_gemfile_lock_entriesAndrew White2016-09-201-0/+2
|\ \ \ | | | | | | | | Restore missing Gemfile.lock entries
| * | | Restore missing Gemfile.lock entriesRyuta Kamizono2016-09-201-0/+2
|/ / / | | | | | | | | | These lines were lost in https://github.com/rails/rails/commit/12d5c210#diff-e79a60dc6b85309ae70a6ea8261eaf95L191.
* | | Merge pull request #26553 from y-yagi/fix_link_to_resqueJon Moss2016-09-191-1/+1
|\ \ \ | | | | | | | | fix link to `resque` [ci skip]
| * | | fix link to `resque` [ci skip]yuuji.yaginuma2016-09-201-1/+1
| | | | | | | | | | | | | | | | | | | | `1-x-stable` branch does not exist, `master` is 1.x branch. Ref: http://words.steveklabnik.com/rescuing-resque-again
* | | | Merge pull request #26552 from rafaelfranca/fix-errors-details-storageRafael França2016-09-193-15/+32
|\ \ \ \ | |/ / / |/| | | Always store errors details information with symbols
| * | | Always store errors details information with symbolsRafael Mendonça França2016-09-193-15/+32
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the association is autosaved we were storing the details with string keys. This was creating inconsistency with other details that are added using the `Errors#add` method. It was also inconsistent with the `Errors#messages` storage. To fix this inconsistency we are always storing with symbols. This will cause a small breaking change because in those cases the details could be accessed as strings keys but now it can not. The reason that we chose to do this breaking change is because `#details` should be considered a low level object like `#messages` is. Fix #26499. [Rafael Mendonça França + Marcus Vieira]
* | | Merge pull request #26447 from kamipo/reduce_array_allocationSantiago Pastorino2016-09-191-3/+1
|\ \ \ | | | | | | | | Reduce array allocation when `where` with passed hash
| * | | Reduce array allocation when `where` with passed hashRyuta Kamizono2016-09-101-3/+1
| | | | | | | | | | | | | | | | | | | | In most case `where` is called with passed hash. In the case initializing `binds` is unnecessary.
* | | | Merge pull request #26543 from y-yagi/use_rails_command_in_restart_testKasper Timm Hansen2016-09-191-7/+7
|\ \ \ \ | | | | | | | | | | use rails command in restart task test
| * | | | use rails command in restart task testyuuji.yaginuma2016-09-191-7/+7
| | | | |
* | | | | Merge pull request #26448 from kamipo/remove_collection_association_uniqKasper Timm Hansen2016-09-181-1/+0
|\ \ \ \ \ | |_|_|_|/ |/| | | | Remove unnecessry `alias uniq distinct` for collection association
| * | | | Remove unnecessry `alias uniq distinct` for collection associationRyuta Kamizono2016-09-101-1/+0
| | |/ / | |/| | | | | | | | | | `CollectionAssociation` is internal class and `uniq` is not called.
* | | | Merge pull request #26534 from Neodelf/new_branch_nameJon Moss2016-09-177-7/+7
|\ \ \ \ | | | | | | | | | | [ci skip] Fixed commas according to Oxford comma in rdoc and guides
| * | | | [ci skip] Fixed commas according to Oxford comma in rdoc and guidesAndrey Molchanov2016-09-177-7/+7
| | | | |
* | | | | Merge pull request #26446 from kamipo/rename_type_var_name_to_typeEileen M. Uchitelle2016-09-171-10/+8
|\ \ \ \ \ | | | | | | | | | | | | Rename variable name that returning `type_for` to `type` from `column`
| * | | | | Rename variable name that returning `type_for` to `type` from `column`Ryuta Kamizono2016-09-111-10/+8
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | `column_for` was changed to `type_for` to return `type` object at 36bd52b4. But variable name is still `column`. It is very confusing. Rename variable name `column` to `type` for readability.
* | | | | Merge pull request #26497 from koppen/26496-touch_with_optimistic_lockingEileen M. Uchitelle2016-09-173-1/+9
|\ \ \ \ \ | | | | | | | | | | | | Clear attribute changes after handling locking
| * | | | | Clear attribute changes after handling lockingJakob Skjerning2016-09-143-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this the changes to the lock version column will stick around even after `touch` returns. Before: model.touch model.changes # => {"lock_version"=>[0, "1"]} After: model.touch model.changes # {}
* | | | | | Merge pull request #26532 from Neodelf/new_branch_nameप्रथमेश Sonpatki2016-09-171-2/+11
|\ \ \ \ \ \ | | |_|/ / / | |/| | | | [ci skip] Fix bad examples in rdoc
| * | | | | [ci skip] Fix bad examples in rdocAndrey Molchanov2016-09-171-2/+11
|/ / / / /
* | | | | Merge pull request #26528 from y-yagi/fix_formatting_of_fetchप्रथमेश Sonpatki2016-09-171-3/+3
|\ \ \ \ \ | | | | | | | | | | | | fix formatting of `Cache::Store#fetch` [ci skip]
| * | | | | fix formatting of `Cache::Store#fetch` [ci skip]yuuji.yaginuma2016-09-171-3/+3
| | |_|/ / | |/| | | | | | | | | | | | | Single backticks don't work with rdoc.
* | | | | Merge pull request #26523 from maclover7/jm-grammar-26382Rafael França2016-09-161-1/+2
|\ \ \ \ \ | |/ / / / |/| | | | Fix small grammar issue introduced in #26382
| * | | | Fix small grammar issue introduced in #26382Jon Moss2016-09-161-1/+2
|/ / / / | | | | | | | | | | | | [ci skip]
* | | | Merge pull request #26516 from grosser/grosser/assertRafael França2016-09-1675-362/+362
|\ \ \ \ | | | | | | | | | | improve error message when include assertions fail
| * | | | improve error message when include assertions failMichael Grosser2016-09-1675-362/+362
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | assert [1, 3].includes?(2) fails with unhelpful "Asserting failed" message assert_includes [1, 3], 2 fails with "Expected [1, 3] to include 2" which makes it easier to debug and more obvious what went wrong
* | | | | Merge pull request #26514 from louim/patch-1Rafael França2016-09-161-8/+8
|\ \ \ \ \ | | | | | | | | | | | | Update ActiveModel::Dirty Doc to be simpler to understand
| * | | | | Update ActiveModel::Dirty Doc [ci skip]Louis-Michel Couture2016-09-161-8/+8
| |/ / / /
* | | | | Merge pull request #26509 from claudiob/rename-asRafael França2016-09-161-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Rename test to match what it does
| * | | | | Rename test to match what it doesclaudiob2016-09-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit d270da569 changed the `form_for` API from `:object_name` to `:as`. It also change the related test body, but not its title, which is changed here. See https://github.com/rails/rails/commit/d270da569efeabd7cd563028816452236713aa9f#diff-52455f1e82acf12551bc5e7e26b82008 I realize this is a small commit but I was having trouble understanding what the test was about since there is no "object_name" in the code. I imagine the same may happen to other developers, therefore this commit.
* | | | | | Merge pull request #26507 from Neodelf/actionview_duplicateRafael França2016-09-162-6/+2
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | DRYing duplicate methods
| * | | | | DRYing duplicate methodsAndrey Molchanov2016-09-162-6/+2
| | | | | |
* | | | | | Force correct namespace when using TransactionManagerArthur Neves2016-09-161-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | [fixes #26441]
* | | | | | Merge pull request #26510 from y-yagi/fix_typo_in_add_column_docJon Moss2016-09-151-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | fix typo in `add_column` doc [ci skip]
| * | | | | | fix typo in `add_column` doc [ci skip]yuuji.yaginuma2016-09-161-1/+1
| | | | | | |
* | | | | | | Merge pull request #26508 from louim/patch-1Jon Moss2016-09-151-1/+1
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Update ActiveModel::Dirty Doc [ci skip]
| * | | | | | Update ActiveModel::Dirty Doc [ci skip]Louis-Michel Couture2016-09-151-1/+1
|/ / / / / / | | | | | | | | | | | | Fix potentially misleading example.
* | | | | | bug report template for migrations (#26488)Girish Sonawane2016-09-153-0/+130
| | | | | | | | | | | | | | | | | | * added bug report template for migrations
* | | | | | Merge pull request #26469 from y-yagi/remove_useless_optionRafael França2016-09-141-5/+0
|\ \ \ \ \ \ | | | | | | | | | | | | | | remove useless `gemfile` option
| * | | | | | remove useless `gemfile` optionyuuji.yaginuma2016-09-121-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rails application for test creates with `--skip-gemfile` option. https://github.com/rails/rails/blob/master/railties/test/isolation/abstract_unit.rb#L333 Therefore, regardless of the option, Gemfile is not created.
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-05 21:05:39 +0000