aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Active Model: Improve CHANGELOG and documentation for ↵Prathamesh Sonpatki2016-05-052-12/+23
| | | | | | | | | | | `validates_acceptance_of` [ci skip] - Improve CHANGELOG entry for #18439. - The documentation is updated as per changes in PR #18439 to the `accept` option. - The explanation about the virtual attribute is moved at the end so that the arity of `accept` option is explained first. - Added a note that `message` can also be passed to `validates_acceptance_of`.
* add #24385 to 5.0 release notes [ci skip]yuuji.yaginuma2016-04-291-0/+7
|
* rm unused requireVipul A M2016-04-291-1/+0
| | | | Signed-off-by: Jeremy Daer <jeremydaer@gmail.com>
* Extract `add_sql_comment!` methodRyuta Kamizono2016-04-293-20/+13
| | | | | | Refactor of #22911. Signed-off-by: Jeremy Daer <jeremydaer@gmail.com>
* Treat blank comments as no comment for indexesRyuta Kamizono2016-04-293-6/+11
| | | | | | Follow up of 1683410. Signed-off-by: Jeremy Daer <jeremydaer@gmail.com>
* Migrations: move version-finding responsibilityyui-knk2016-04-292-7/+11
| | | | | | | `ActiveRecord::Migration` needn't know about migration version compatibility lookup. Delegate it to the Compatibility module. Signed-off-by: Jeremy Daer <jeremydaer@gmail.com>
* restores the regexp used in String#blank?Xavier Noria2016-04-291-4/+3
| | | | | | | | | | This commit undoes 54243fe. Reason: Further investigation has shown the benefit is not so clear generally speaking. There is a long discussion and several benchmarks in the PR #24658 if you are interested in the details.
* Chomp: prefer String#chomp where we can for a clarity boostlvl0nax2016-04-293-7/+5
| | | | | | Closes #24766, #24767 Signed-off-by: Jeremy Daer <jeremydaer@gmail.com>
* AS::Duration changelog polishJeremy Daer2016-04-291-15/+13
| | | | | | | | Use the updated changelog from the first merge: 7254517 References #22806, #24762. [ci skip]
* Merge pull request #24788 from connorshea/patch-1Jon Moss2016-04-291-52/+52
|\ | | | | Update configuring.md with colons and periods
| * Update configuring.md with colons and periodsConnor Shea2016-04-291-52/+52
|/ | | Some configuration options started with a capital letter indicating the beginning of a sentence, which didn't makes sense without the colons. Similarly, some were missing periods at the end of their descriptions. [ci skip]
* Merge pull request #24787 from connorshea/patch-1Rafael França2016-04-292-3/+3
|\ | | | | Patch 1
| * Consistent spelling of "gzipped" [ci skip]Connor Shea2016-04-291-1/+1
| |
| * Fix typos in asset_pipeline.md [ci skip]Connor Shea2016-04-291-2/+2
|/ | | Per discussion in #24786.
* Merge pull request #24786 from vipulnsward/document-gzip-assetsRichard Schneeman2016-04-292-0/+13
|\ | | | | Document config.assets.gzip flag.
| * Document config.assets.gzip flag. Follow up of ↵Vipul A M2016-04-292-0/+13
| | | | | | | | | | | | https://github.com/rails/sprockets-rails/pull/342. [ci skip]
* | Merge branch 'master' of github.com:rails/docrailsVijay Dev2016-04-298-25/+31
|\ \ | | | | | | | | | | | | Conflicts: guides/source/configuring.md
| * | s/statment/statement/Vipul A M2016-04-241-1/+1
| | |
| * | be specific when the debug_exception_response_format to :apiVipul A M2016-04-231-1/+1
| | | | | | | | | | | | [ci skip]
| * | Mention default values for `debug_exception_response_format` in configuring ↵Prathamesh Sonpatki2016-04-221-1/+1
| | | | | | | | | | | | guide [ci skip]
| * | `debug_exception_format` is set to :api by default for API apps only, not ↵Prathamesh Sonpatki2016-04-221-1/+1
| | | | | | | | | | | | | | | | | | for normal apps. [ci skip]
| * | [ci skip] Aline results of code examples in commentsyui-knk2016-04-201-2/+2
| | |
| * | s/determins/determines/Vipul A M2016-04-201-1/+1
| | | | | | | | | | | | [ci skip]
| * | - Be consistent in providing file locations of schema, model and initializerVipul A M2016-04-131-1/+6
| | | | | | | | | | | | [ci skip]
| * | - Make sure we end sentencesVipul A M2016-04-131-5/+5
| | | | | | | | | | | | | | | | | | - Change from "The Task" to "The Update Task" as referred elsewhere [ci skip]
| * | [ci skip] Add some missing commasPrathamesh Sonpatki2016-04-121-2/+2
| | |
| * | [ci skip] Typo fix. `loggs` -> `logs`Prathamesh Sonpatki2016-04-121-1/+1
| | |
| * | - Fix typo in config guide, from config.logger sectionVipul A M2016-04-101-3/+3
| | | | | | | | | | | | | | | | | | - Make bulleted lists, end properly [ci skip]
| * | s/Thoughtbot/thoughtbot/Vipul A M2016-04-101-1/+1
| | | | | | | | | | | | | | | | | | See https://github.com/thoughtbot/presskit/blob/master/README.md#name for name capitalization. [ci skip]
| * | [ci skip] Fix formatting of code snippet in Action Cable guidePrathamesh Sonpatki2016-04-051-7/+8
| | |
| * | [ci skip] Update code used for filtering passwords in the configuring guidePrathamesh Sonpatki2016-04-051-1/+1
| | |
* | | Merge pull request #24776 from prathamesh-sonpatki/aj-release-notesKasper Timm Hansen2016-04-292-1/+22
|\ \ \ | |_|/ |/| | Active Job: Add note about ability to configure adapters on per job basis in CHANGELOG and release notes
| * | Active Job: Add note about ability to configure adapters on per job basis in ↵Prathamesh Sonpatki2016-04-292-1/+22
| | | | | | | | | | | | | | | | | | CHANGELOG and release notes. [ci skip]
* | | Merge pull request #24762 from Envek/solid_durations_2Andrew White2016-04-294-11/+47
|\ \ \ | | | | | | | | [FIX] Change 1.week to create 1 week durations instead of 7 days durations.
| * | | Change 1.week to create 1 week durations instead of 7 days durations.Andrey Novikov2016-04-284-11/+47
| | | | | | | | | | | | | | | | This is just to remove astonishment from getting `3600 seconds` from typing `1.hour`.
* | | | Merge pull request #24781 from maclover7/rm-envGuillermo Iguaran2016-04-283-12/+39
|\ \ \ \ | | | | | | | | | | Remove last uses of `@env[]` and `@env[]=`
| * | | | Remove last uses of `@env[]` and `@env[]=`Jon Moss2016-04-283-12/+39
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Last August (2015), @tenderlove worked to remove all `@env[]` and `@env[]=`, in favor of using `set_header`, `get_header`, etc. (Here's an [example commit](https://github.com/rails/rails/commit/f16a33b68efc3dc57cfafa27651b9a765e363fbf)). This PR should remove the last uses of these methods, and fully convert them to the newly standardized API.
* | | | Merge pull request #24653 from kbrock/kinder_locking_columnRafael França2016-04-282-1/+54
|\ \ \ \ | | | | | | | | | | schema_load triggers 2nd schema_load (via locking)
| * | | | test the number of times the schema is loadingKeenan Brock2016-04-281-0/+53
| | | | |
| * | | | schema_load triggers 2nd schema_load (via locking)Keenan Brock2016-04-281-1/+1
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, loading the schema (schema_load) accesses the locking column (locking_column) which defaults the value (reset_locking_column) which invalidates the schema (reload_schema_from_cache) which forces another schema load. Good news: The second schema_load does accesses locking_column, but locking_column is set, so it does not reset_locking_column and it does not trigger an infinite loop. The solution is not invalidate the cache while default locking_column
* | | | Merge pull request #24770 from vipulnsward/guieds-typeRafael França2016-04-281-1/+1
|\ \ \ \ | |/ / / |/| | | Pass over debugging guide
| * | | Pass over debugging guideVipul A M2016-04-281-1/+1
| | | | | | | | | | | | | | | | [ci skip]
* | | | Merge pull request #24772 from javiervidal/broken_links_ryandaigleKasper Timm Hansen2016-04-282-14/+14
|\ \ \ \ | |/ / / |/| | | Fix broken links to ryandaigle.com [ci skip]
| * | | Fix broken links to ryandaigle.com [ci skip]Javier Vidal2016-04-282-14/+14
|/ / /
* | | Merge pull request #24764 from deivid-rodriguez/review_debugging_guideVipul A M2016-04-281-140/+185
|\ \ \ | | | | | | | | Review the `byebug` debugging guide [ci skip]
| * | | Review the `byebug` debugging guideDavid Rodríguez2016-04-281-140/+185
|/ / / | | | | | | | | | | | | | | | | | | | | | * Several formatting fixes. * Updated command outputs to latest version. * Minor rewordings. [ci skip]
* | | Merge pull request #24765 from ↵Vipul A M2016-04-281-4/+4
|\ \ \ | | | | | | | | | | | | | | | | ralinchimev/fix-inconsistencies-in-rails-documentation-generate-controller-examples Fix inconsistent usage of a capital letter in rails generate examples [ci skip]
| * | | Fix inconsistent usage of capital letter in rails generate command examplesRalin Chimev2016-04-281-4/+4
|/ / /
* | | Merge pull request #24761 from rubys/patch-3Richard Schneeman2016-04-271-1/+1
|\ \ \ | |/ / |/| | Update to the Rails 5 version of the book [ci skip]
| * | Update to the Rails 5 version of the bookSam Ruby2016-04-271-1/+1
|/ /