aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Allow irb options to be passed from `rails console` commandyuuji.yaginuma2017-05-083-6/+32
| | | | Fixes #28988
* Merge pull request #29005 from kamipo/should_escape_meta_chars_in_regexpKasper Timm Hansen2017-05-0730-118/+118
|\ | | | | Should escape meta characters in regexp
| * Should escape meta characters in regexpRyuta Kamizono2017-05-0730-118/+118
|/
* Merge pull request #29002 from kamipo/fix_warning_ambiguous_first_argumentMatthew Draper2017-05-071-2/+2
|\ | | | | Fix `warning: ambiguous first argument`
| * Fix `warning: ambiguous first argument`Ryuta Kamizono2017-05-061-2/+2
|/ | | | | | | | | | | | | | | | ``` % ARCONN=sqlite3 be ruby -w -Itest test/cases/quoting_test.rb test/cases/quoting_test.rb:92: warning: ambiguous first argument; put parentheses or a space even after `/' operator test/cases/quoting_test.rb:96: warning: ambiguous first argument; put parentheses or a space even after `/' operator Using sqlite3 Run options: --seed 9495 ..................................... Finished in 0.046403s, 797.3622 runs/s, 1120.6172 assertions/s. 37 runs, 52 assertions, 0 failures, 0 errors, 0 skips ```
* Merge pull request #28989 from matthewd/quoted_id-deprecationMatthew Draper2017-05-062-2/+22
|\ | | | | Clarify deprecation message for #quoted_id
| * Clarify deprecation message for #quoted_idMatthew Draper2017-05-052-2/+22
| | | | | | | | | | In this case, it's the method definition that's more at fault, rather than the current caller.
* | Merge pull request #28999 from kamipo/remove_unused_delegation_test_call_methodRafael França2017-05-051-27/+7
|\ \ | | | | | | Remove unused `DelegationTest#call_method`
| * | Remove unused `DelegationTest#call_method`Ryuta Kamizono2017-05-061-27/+7
|/ / | | | | | | `DelegationTest#call_method` is no longer used since 9d79334a.
* | Merge pull request #28981 from kamipo/dont_use_arel_engineMatthew Draper2017-05-054-10/+7
|\ \ | | | | | | Don't pass `arel.engine` to `Arel::SelectManager.new`
| * | Don't pass `arel.engine` to `Arel::SelectManager.new`Ryuta Kamizono2017-05-054-10/+7
|/ / | | | | | | | | | | The argument of `Arel::SelectManager.new` is `table`, not `engine`. https://github.com/rails/arel/blob/v8.0.0/lib/arel/select_manager.rb#L10
* | Merge pull request #28986 from tjschuck/delegate_missing_to_doc_fixesRafael França2017-05-042-28/+23
|\ \ | | | | | | Assorted delegate_missing_to doc fixes
| * | Update test names to match method nameT.J. Schuck2017-05-041-6/+6
| | | | | | | | | The method is named `delegate_missing_to`, not `delegate_to_missing`
| * | Assorted delegate_missing_to doc fixesT.J. Schuck2017-05-041-22/+17
| |/ | | | | | | | | | | | | | | | | | | | | | | * Fix rdoc code formatting — `tt`, not backticks * Fix/simplify sentence grammar — should at least just be “and the like”, not “likes”, but this is just general tightening up. * Add note that delegated methods must be public. Tested here: https://github.com/rails/rails/blob/7ff5ccae94ce2aff76b5f8a31a28e305a047d642/activesupport/test/core_ext/module_test.rb#L359-L365 * Simplify example code for delegate_missing_to. The example had complexity that wasn’t necessary for demonstrating `delegate_missing_to`. This gets rid of a bunch of cruft so the example is more obvious about what’s going on regarding the feature itself. [ci skip]
* | Merge pull request #28983 from ↵Rafael França2017-05-041-11/+3
|\ \ | | | | | | | | | | | | kamipo/remove_useless_target_records_from_association Remove useless `target_records_from_association`
| * | Remove useless `target_records_from_association`Ryuta Kamizono2017-05-041-11/+3
| | | | | | | | | | | | Since through association is always loaded by `preloader.preload`.
* | | Merge pull request #28984 from yahonda/mysql2_046Rafael França2017-05-041-3/+3
|\ \ \ | |_|/ |/| | Use mysql2 0.4.6 to suport MySQL 8.0.1
| * | Use mysql2 0.4.6 to suport MySQL 8.0.1Yasuo Honda2017-05-041-3/+3
| | | | | | | | | | | | Follow up #28733 and brianmario/mysql2#840
* | | Avoid rubygems 2.6.12 for nowMatthew Draper2017-05-041-1/+1
| | | | | | | | | | | | | | | It's causing a test to fail, and we're not yet sure what (if anything) we should be doing differently. See rubygems/rubygems#1911.
* | | Merge pull request #28979 from y-yagi/build_package_json_in_create_root_filesRafael França2017-05-031-4/+5
|\ \ \ | | | | | | | | Move `package.json` creation to `create_root_files`
| * | | Move `package.json` creation to `create_root_files`yuuji.yaginuma2017-05-041-4/+5
|/ / / | | | | | | | | | | | | Since `package.json` is created in root directory, it is appropriate to create it in `create_root_files`.
* | | Merge pull request #28337 from riseshia/patch-1Rafael França2017-05-031-1/+1
|\ \ \ | | | | | | | | Fix to use correct path with language on guide
| * | | Fix to use correct path with language on guideShia2017-03-081-1/+1
| | | |
* | | | Merge pull request #28978 from riseshia/patch-2Rafael França2017-05-031-1/+1
|\ \ \ \ | |_|_|/ |/| | | Use target language's document.yaml [ci skip]
| * | | Use target language's document.yaml [ci skip]Shia2017-05-041-1/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Let me build guide with "ko" language, Before: - It try to use guides/source/documents.yaml After: - It try to use guides/source/ko/documents.yaml
* | | Merge pull request #28976 from kamipo/should_test_bound_attributesRafael França2017-05-031-8/+12
|\ \ \ | | | | | | | | Should test against `Relation#bound_attributes`
| * | | Should test against `Relation#bound_attributes`Ryuta Kamizono2017-05-041-8/+12
|/ / / | | | | | | | | | Since legacy `Relation#bind_values` was removed in b06f64c.
* | | Merge pull request #28974 from kamipo/extract_bind_param_and_bind_attributeRafael França2017-05-0311-51/+38
|\ \ \ | | | | | | | | Extract `bind_param` and `bind_attribute` into `ActiveRecord::TestCase`
| * | | Extract `bind_param` and `bind_attribute` into `ActiveRecord::TestCase`Ryuta Kamizono2017-05-0411-51/+38
| | | | | | | | | | | | | | | | These are used in tests from anywhere.
* | | | Merge pull request #28975 from ↵Rafael França2017-05-031-9/+0
|\ \ \ \ | |/ / / |/| | | | | | | | | | | kamipo/remove_duplicated_test_merging_reorders_bind_params Remove duplicated `test_merging_reorders_bind_params`
| * | | Remove duplicated `test_merging_reorders_bind_params`Ryuta Kamizono2017-05-041-9/+0
|/ / / | | | | | | | | | | | | | | | The same test exists in `test/cases/relation/merging_test.rb`. https://github.com/rails/rails/blob/v5.1.0/activerecord/test/cases/relation/merging_test.rb#L94-L101
* | | Merge pull request #28967 from stve/active-model-validator-kind-examplesRyuta Kamizono2017-05-031-2/+2
|\ \ \ | | | | | | | | fix ActiveModel::Validator#kind code examples
| * | | fix ActiveModel::Validator#kind code examples [ci skip]stve2017-05-021-2/+2
|/ / /
* | | Merge pull request #28966 from stve/active-model-uniqueness-validatorsGuillermo Iguaran2017-05-022-5/+4
|\ \ \ | | | | | | | | remove uniqueness validators from ActiveModel examples [ci-skip]
| * | | remove uniqueness validators from ActiveModel examplesstve2017-05-022-5/+4
| | | |
* | | | Merge pull request #28961 from maclover7/jm-fix-28959Rafael França2017-05-025-3/+13
|\ \ \ \ | |/ / / |/| | | Add type caster to `RuntimeReflection#alias_name`
| * | | Add type caster to `RuntimeReflection#alias_name`Jon Moss2017-05-025-3/+13
| | |/ | |/| | | | | | | | | | Since we have been using this `Arel::Table` since 111ccc832bc977b15af12c14e7ca078dad2d4373, in order to properly handle queries, it's important that we properly type cast arguments.
* | | Merge pull request #28835 from fphilipe/masterRafael França2017-05-021-0/+1
|\ \ \ | |/ / |/| | Add missing require in ActionMailer::Base
| * | Add missing require in ActionMailerPhilipe Fatio2017-05-021-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Without this, Action Mailer doesn't work like it used to on version 4. The following snippet fails since version 5: > require 'action_mailer' > ActionMailer::Base NameError: uninitialized constant ActiveSupport::Rescuable
* | | Merge pull request #28956 from bdewater/patch-1Kasper Timm Hansen2017-05-021-1/+1
|\ \ \ | | | | | | | | Rails secrets is not built on but inspired by Sekrets
| * | | Rails secrets is not built on but inspired by SekretsBart2017-05-021-1/+1
|/ / / | | | | | | Put wording in line with http://weblog.rubyonrails.org/2017/4/27/Rails-5-1-final/
* | | Merge pull request #28938 from kamipo/deprecate_supports_statement_cacheRafael França2017-05-016-22/+12
|\ \ \ | | | | | | | | Deprecate `supports_statement_cache?`
| * | | Deprecate `supports_statement_cache?`Ryuta Kamizono2017-05-016-22/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `supports_statement_cache?` was introduced in 3.1.0.beta1 (104d0b2) for bind parameter substitution, but it is no longer used in 3.1.0.rc1 (73ff679). Originally it should respect `prepared_statements` rather than `supports_statement_cache?` (fd39847). One more thing, named `supports_statement_cache?` is pretty misreading. We have `StatementCache` and `StatementPool`. However, `supports_statement_cache?` doesn't mean `StatementCache`, but `StatementPool` unlike its name. https://github.com/rails/rails/blob/v5.1.0/activerecord/lib/active_record/statement_cache.rb https://github.com/rails/rails/blob/v5.1.0/activerecord/lib/active_record/connection_adapters/statement_pool.rb
* | | | Merge pull request #28930 from kamipo/use_flat_mapRafael França2017-05-012-3/+3
|\ \ \ \ | | | | | | | | | | Use `flat_map` rather than `map(&:...).flatten`
| * | | | Use `flat_map` rather than `map(&:...).flatten`Ryuta Kamizono2017-04-292-3/+3
| | | | |
* | | | | Merge pull request #28943 from koshigoe/port-arg-has-precedence-over-env-varRafael França2017-05-012-2/+15
|\ \ \ \ \ | | | | | | | | | | | | CLI arg `--port` has precedence over env `PORT`
| * | | | | CLI arg `--port` has precedence over env `PORT`.koshigoe2017-05-012-2/+15
| | |/ / / | |/| | |
* | | | | Merge pull request #28753 from st0012/add-render-with-variants-testKasper Timm Hansen2017-05-013-0/+10
|\ \ \ \ \ | | | | | | | | | | | | Add tests for rendering with variants
| * | | | | Add tests for render template/partial with variants option.Stan Lo2017-04-143-0/+10
| | | | | |
* | | | | | Merge pull request #28939 from kirs/maintain_test_schema_friendlyAndrew White2017-05-011-1/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Friendly maintain schema error in test runner
generated by cgit v1.2.3 (git 2.25.1) at 2024-07-10 01:16:45 +0000