aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* initialize the right variableRafael Mendonça França2014-06-171-1/+1
|
* :scissors:Rafael Mendonça França2014-06-171-8/+8
|
* Add a generic --skip-gems options to generatorRafael Mendonça França2014-06-173-10/+19
| | | | | | | | | | Also remove --skip-turbolinks. This option is useful if users want to remove some gems like jbuilder, turbolinks, coffee-rails, etc that don't have specific options on the generator. rails new my_app --skip-gems turbolinks coffee-rails
* Merge pull request #15784 from sgrif/sg-delimiterRafael Mendonça França2014-06-174-7/+54
|\ | | | | Ensure `OID::Array#type_cast_for_database` matches PG's quoting behavior
| * Ensure `OID::Array#type_cast_for_database` matches PG's quoting behaviorSean Griffin2014-06-174-7/+54
|/ | | | | | Also takes a step towards supporting types which use a character other than ',' for the delimiter (`box` is the only built in type for which this is the case)
* Merge pull request #15778 from sgrif/sg-pg-mutable-arraysMatthew Draper2014-06-184-38/+62
|\ | | | | Move pg array database type casting to the Array type
| * Move array database type casting to the Array typeSean Griffin2014-06-174-38/+62
| | | | | | | | | | | | | | | | | | The case where we have a column object, but don't have a type cast method involves type casting the default value when changing the schema. We get one of the column definition structs instead. That is a case that I'm trying to remove overall, but in the short term, we can achieve the same behavior without needing to pass the adapter to the array type by creating a fake type that proxies to the adapter.
* | Merge pull request #15716 from schuetzm/skip-turbolinksRafael Mendonça França2014-06-172-2/+21
|\ \ | | | | | | Add an option to skip installation of Turbolinks
| * | Add an option to skip installation of TurbolinksMarc Schütz2014-06-142-2/+21
| | |
* | | Merge pull request #15727 from aditya-kapoor/add-tests-for-migrationMatthew Draper2014-06-181-0/+17
|\ \ \ | | | | | | | | Add test cases for Migration#inverse_of
| * | | Add test cases for Migration#inverse_ofAditya Kapoor2014-06-171-0/+17
| | | |
* | | | Fix documentation from #15669 [ci skip]Zachary Scott2014-06-171-3/+5
| | | |
* | | | [ci skip] fix doc for the Hash#assert_valid_keysAditya Kapoor2014-06-171-3/+3
| | | |
* | | | Merge pull request #15780 from sgrif/sg-dont-always-save-mutable-typesRafael Mendonça França2014-06-173-2/+4
|\ \ \ \ | | | | | | | | | | Don't assume that Hstore columns have always changed
| * | | | Don't assume that Hstore columns have always changedSean Griffin2014-06-173-2/+4
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | HStore columns come back from the database separated by a comma and a space, not just a comma. We need to mirror that behavior since we compare the two values. Also adds a regression test against JSON to ensure we don't have the same bug there.
* | | | Merge pull request #15775 from sgrif/sg-use-column-defaults-in-dirtyRafael Mendonça França2014-06-171-11/+3
|\ \ \ \ | |/ / / |/| | | Use `column_defaults` in dirty for checking changed defaults
| * | | Use `column_defaults` in dirty for checking changed defaultsSean Griffin2014-06-171-11/+3
|/ / / | | | | | | | | | | | | | | | We no longer need to "init changed attributes" from the initializer, either, as there is no longer a case where a given value would differ from the default, but would not already be marked as changed.
* | | Merge pull request #15771 from sgrif/sg-stop-messing-with-column-defaultsRafael Mendonça França2014-06-173-12/+32
|\ \ \ | | | | | | | | Don't mess with `column_defaults` when optimistic locking is enabled
| * | | Don't mess with `column_defaults` when optimistic locking is enabledSean Griffin2014-06-173-12/+32
|/ / /
* | | Merge pull request #15768 from zuhao/refactor_activesupport_concern_testYves Senn2014-06-171-1/+2
|\ \ \ | | | | | | | | Move test teardown into `ensure` block.
| * | | Move test teardown into `ensure` block.Zuhao Wan2014-06-171-1/+2
|/ / /
* | | add `bin/rake db:purge` task to empty the current database.Yves Senn2014-06-174-0/+55
| | |
* | | fix typo in test method names. [ci skip]Yves Senn2014-06-171-4/+4
| | |
* | | Merge pull request #15765 from rahul100885/masterYves Senn2014-06-171-3/+3
|\ \ \ | | | | | | | | [ci skip] doc corrected : ActiveRecord::Base.connections to ActiveRecord::Base.configurations
| * | | [ci skip] doc corrected : ActiveRecord::Base.connections to ↵Rahul P. Chaudhari2014-06-171-3/+3
|/ / / | | | | | | | | | ActiveRecord::Base.configurations
* | | Formatting fix in AS changelog [ci skip]Juanjo Bazán2014-06-161-2/+5
| | |
* | | Merge pull request #15760 from sgrif/sg-decorate-matchingRafael Mendonça França2014-06-166-32/+57
|\ \ \ | | | | | | | | Promote time zone aware attributes to a first class type decorator
| * | | Promote time zone aware attributes to a first class type decoratorSean Griffin2014-06-166-32/+57
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This refactoring revealed the need for another form of decoration, which takes a proc to select which it applies to (There's a *lot* of cases where this form can be used). To avoid duplication, we can re-implement the old decoration in terms of the proc-based decoration. The reason we're `instance_exec`ing the matcher is for cases such as time zone aware attributes, where a decorator is defined in a parent class, and a method called in the matcher is overridden by a child class. The matcher will close over the parent, and evaluate in its context, which is not the behavior we want.
* | | Merge pull request #15759 from sgrif/sg-extract-type-decoratorRafael Mendonça França2014-06-161-8/+24
|\ \ \ | | | | | | | | Refactor application of type decorators into an object
| * | | Refactor application of type decorators into an objectSean Griffin2014-06-161-8/+24
| | | |
* | | | Merge pull request #15758 from aditya-kapoor/replace-keywordRafael Mendonça França2014-06-162-4/+4
|\ \ \ \ | | | | | | | | | | [ci skip] /javascript/ ~> JavaScript
| * | | | [ci skip] /javascript/ ~> JavaScriptAditya Kapoor2014-06-172-4/+4
|/ / / /
* | | | Merge pull request #15756 from aditya-kapoor/add-api-doc-for-queryingZachary Scott2014-06-161-1/+9
|\ \ \ \ | |/ / / |/| | | [ci skip] add API doc for AR Group.
| * | | [ci skip] add API doc for AR Group.Aditya Kapoor2014-06-171-1/+9
|/ / /
* | | :scissors: and :nail_care: from d60c405 [ci skip]Zachary Scott2014-06-161-3/+3
| | |
* | | Merge pull request #15751 from aditya-kapoor/correct-serialization-docZachary Scott2014-06-161-11/+11
|\ \ \ | | | | | | | | [ci skip] correct doc for serialization in accordance with API Doc Guide
| * | | [ci skip] correct doc for serialization in accordance with API Doc GuideAditya Kapoor2014-06-161-11/+11
| | | |
* | | | Merge pull request #15755 from sgrif/sg-dirty-refactorRafael Mendonça França2014-06-163-9/+26
|\ \ \ \ | | | | | | | | | | Refactor in-place dirty checking to use the attribute object
| * | | | Refactor in-place dirty checking to use the attribute objectSean Griffin2014-06-163-9/+26
| | | | |
* | | | | Add important plugins as one of the tasks of the releaseRafael Mendonça França2014-06-161-0/+12
|/ / / / | | | | | | | | | | | | [ci skip]
* | | | Merge pull request #15684 from aditya-kapoor/add-tests-for-assert-valid-keysRafael Mendonça França2014-06-161-0/+15
|\ \ \ \ | | | | | | | | | | Add test cases for Hash#asset_valid_keys
| * | | | Add test cases for Hash#asset_valid_keysAditya Kapoor2014-06-141-0/+15
| | | | |
* | | | | Merge pull request #13963 from lucas-clemente/pending_migrationsRafael Mendonça França2014-06-162-5/+58
|\ \ \ \ \ | |_|/ / / |/| | | | Skip migration check if adapter doesn't support it
| * | | | skip migration check if adapter doesn't support itLucas Clemente2014-04-192-5/+58
| | | | |
* | | | | Merge pull request #15744 from mmozuras/special_keys_setYves Senn2014-06-161-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Change Http::Cache::SPECIAL_KEYS from Array to Set
| * | | | | Change Http::Cache::SPECIAL_KEYS from Array to SetMindaugas Mozūras2014-06-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Slightly improves performance, for example, a simple benchmark: ```ruby require 'benchmark/ips' require 'set' SPECIAL_KEYS = %w[extras no-cache max-age public must-revalidate] SPECIAL_KEYS_SET = Set.new(SPECIAL_KEYS) directive = 'must-revalidate' Benchmark.ips do |x| x.report('array') { SPECIAL_KEYS.include?(directive) } x.report('set') { SPECIAL_KEYS_SET.include?(directive) } end ``` Output: ``` ------------------------------------- array 67926 i/100ms set 74054 i/100ms ------------------------------------- array 2318423.4 (±2.8%) i/s - 11615346 in 5.014899s set 3387981.8 (±4.7%) i/s - 16958366 in 5.019355s ```
* | | | | | Merge pull request #15743 from tgxworld/remove_unused_parametersYves Senn2014-06-161-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Remove unused parameter.
| * | | | | | Remove unused parameter.Guo Xiang Tan2014-06-151-1/+1
| |/ / / / /
* | | | | | Merge pull request #15722 from akshay-vishnoi/spell-correctZachary Scott2014-06-158-11/+11
|\ \ \ \ \ \ | | | | | | | | | | | | | | [ci skip] /mysql/i -> MySQL, Spell correct in continuation to #15555
| * | | | | | /mysql/i -> MySQL, Spell correct in continuation to #15555Akshay Vishnoi2014-06-148-11/+11
| | | | | | |