aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | `@previous_reflection.options[:source_type]` in `PolymorphicReflection` is ↵Ryuta Kamizono2017-08-281-10/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | always true Because `add_as_polymorphic_through` is only called when `options[:source_type]` is true.
* | | | | | | | | Merge pull request #30421 from y-yagi/fix_30405Eileen M. Uchitelle2017-08-273-13/+39
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Retrive screenshot in relative path of current directory
| * | | | | | | | | Retrive screenshot in relative path of current directoryyuuji.yaginuma2017-08-273-13/+39
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In Rails engine `Rails.root `returns the path of the dummy application. Therefore, there is no `tmp` directly where the test is running, so can not get the screenshot. For this reason, instead of directly specifying tmp, retrive screenshot by relative path from the current directory. Fixes #30405
* | | | | | | | | Merge pull request #30435 from yhirano55/fix_grammer_in_guide_for_javascriptVipul A M2017-08-271-6/+6
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix grammar in JavaScript in Rails [ci skip]
| * | | | | | | | | Fix grammar in guide [ci skip]Yoshiyuki Hirano2017-08-281-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * I've fixed grammar problems in the guide about JavaScript in Rails.
* | | | | | | | | | Remove unused `RuntimeReflection#alias_candidate`Ryuta Kamizono2017-08-281-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `RuntimeReflection#alias_candidate` is no longer used since 0408e212.
* | | | | | | | | | Remove trailing spaces [ci skip]Ryuta Kamizono2017-08-276-10/+10
|/ / / / / / / / /
* | | | | | | | | Should be appear deprecation message for every call (#29649)Ryuta Kamizono2017-08-272-8/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Context: https://github.com/rails/rails/pull/29619#discussion_r125158589
* | | | | | | | | Omit the default limit for float columns (#28041)Ryuta Kamizono2017-08-273-5/+10
| | | | | | | | |
* | | | | | | | | Prefer to place a table options before `force: :cascade` (#28005)Ryuta Kamizono2017-08-275-11/+10
| |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I was added a table options after `force: :cascade` in #17569 for not touching existing tests (reducing diff). But `force: :cascade` is not an important information. So I prefer to place a table options before `force: :cascade`.
* | | | | | | | Merge pull request #30432 from yhirano55/use_tt_in_doc_for_actionviewRyuta Kamizono2017-08-275-24/+24
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Use tt in doc for ActionView [ci skip]
| * | | | | | | | Use tt in doc for ActionView [ci skip]Yoshiyuki Hirano2017-08-275-24/+24
| | | | | | | | |
* | | | | | | | | Merge pull request #30429 from yhirano55/use_tt_in_doc_for_activerecordRyuta Kamizono2017-08-277-22/+22
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Use tt in doc for ActiveRecord [ci skip]
| * | | | | | | | | Use tt in doc for ActiveRecord [ci skip]Yoshiyuki Hirano2017-08-277-22/+22
| |/ / / / / / / /
* | | | | | | | | Merge pull request #30430 from yhirano55/use_tt_in_doc_for_railtiesRyuta Kamizono2017-08-274-7/+7
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | Use tt in doc for ActiveSupport and Railties [skip ci]
| * | | | | | | | Use tt in doc for railties [skip ci]Yoshiyuki Hirano2017-08-274-7/+7
|/ / / / / / / /
* | | | | | | | Merge pull request #30427 from yhirano55/fix_broken_doc_layout_for_action_viewRyuta Kamizono2017-08-2722-22/+22
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix broken doc layout for action_view [ci skip]
| * | | | | | | | Fix broken doc layout for action_view [ci skip]Yoshiyuki Hirano2017-08-2722-22/+22
| | | | | | | | |
* | | | | | | | | Merge pull request #30426 from robin850/remove-top-level-ast-constantsGeorge Claghorn2017-08-261-1/+8
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | Remove Active Storage duplicated classes from the API site
| * | | | | | | | Remove Active Storage duplicated classes from the API siteRobin Dupret2017-08-261-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since cb5af0d7, some classes that are under Active Storage are now part of the API site. However, these classes aren't nested under a definition of the `ActiveStorage` module but rather name-spaced under it like `ActiveStorage::Foo`. Thus, these classes are present both under the ActiveStorage label and at the root of the site's sidebar so we have to strip out duplicates. [ci skip]
* | | | | | | | | Merge branch 'master' of github.com:rails/docrailsVijay Dev2017-08-261-1/+1
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | |
| * | | | | | | | [ci skip] Fix a typoyui-knk2017-08-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We always plural form for `resources` method.
* | | | | | | | | Merge pull request #30424 from ↵George Claghorn2017-08-262-2/+2
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | koic/fix_order_of_active_storage_disk_url_parameters Fix order of Active Storage DiskService URL parameters
| * | | | | | | | Fix order of Active Storage DiskService URL parametersKoichi ITO2017-08-262-2/+2
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | `content_type` parameter is before `disposition` parameter.
* | | | | | | | Merge pull request #30422 from dansteele/patch-3Vipul A M2017-08-261-1/+4
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | Correct wording and adding info [ci skip]
| * | | | | | | Correct wording and adding info [ci skip]Daniel Steele2017-08-261-1/+4
|/ / / / / / / | | | | | | | | | | | | | | I had to do some I was unsure what kind of caching it was performing, and had to do some Googling. Also improves the wording.
* | | | | | | Fix random CI failure due to non-deterministic sorting orderRyuta Kamizono2017-08-261-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | https://travis-ci.org/rails/rails/jobs/268599781#L784
* | | | | | | Merge pull request #30418 from yhirano55/use_tt_in_doc_for_actionpackRyuta Kamizono2017-08-268-17/+17
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Use tt in doc for action_pack [ci skip]
| * | | | | | | Use tt in doc for ActionPack [ci skip]Yoshiyuki Hirano2017-08-268-17/+17
| | | | | | | |
* | | | | | | | Merge pull request #30417 from ↵Ryuta Kamizono2017-08-262-1/+16
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | y-yagi/fix_cant_modify_forzen_string_error_in_display_image Fix `can't modify frozen String` error in `display_image`
| * | | | | | | | Fix `can't modify frozen String` error in `display_image`yuuji.yaginuma2017-08-262-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Without this, `display_image` raises an error as follwing: ``` RuntimeError: can't modify frozen String rails/actionpack/lib/action_dispatch/system_testing/test_helpers/screenshot_helper.rb:72:in `display_image' rails/actionpack/test/dispatch/system_testing/screenshot_helper_test.rb:40:in `block (2 levels) in <class:ScreenshotHelperTest>' ```
* | | | | | | | | Merge pull request #30416 from yhirano55/use_tt_in_doc_for_action_mailerRyuta Kamizono2017-08-261-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Use tt in doc for action_mailer [ci skip]
| * | | | | | | | | Use tt in doc for action_mailer [ci skip]Yoshiyuki Hirano2017-08-261-1/+1
| | |/ / / / / / / | |/| | | | | | |
* | | | | | | | | Merge pull request #30415 from yhirano55/use_typewriter_in_doc_for_action_cableRyuta Kamizono2017-08-261-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Use typewriter in doc for Action Cable [ci skip]
| * | | | | | | | | Use typewriter in doc for Action Cable [ci skip]Yoshiyuki Hirano2017-08-261-1/+1
| |/ / / / / / / /
* | | | | | | | | Merge pull request #30403 from yhirano55/fix_broken_doc_for_active_recordRyuta Kamizono2017-08-267-7/+7
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix broken doc for Active Record [ci skip]
| * | | | | | | | | Fix broken doc for Active Record [ci skip]Yoshiyuki Hirano2017-08-257-7/+7
| | | | | | | | | |
* | | | | | | | | | Merge pull request #30413 from yhirano55/enable_links_in_active_storage_docsGeorge Claghorn2017-08-252-2/+2
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Enable links in Active Storage docs [ci skip]
| * | | | | | | | | | Enable links in Active Storage docs [ci skip]Yoshiyuki Hirano2017-08-262-2/+2
| | |/ / / / / / / / | |/| | | | | | | |
* | | | | | | | | | Merge pull request #30412 from ↵Richard Schneeman2017-08-253-3/+3
|\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | yhirano55/add_nodoc_for_active_record_attribute_methods Add :nodoc: for ActiveRecord::AttributeMethods [ci skip]
| * | | | | | | | | Add :nodoc: for ActiveRecord::AttributeMethods [ci skip]Yoshiyuki Hirano2017-08-263-3/+3
|/ / / / / / / / /
* | | | | | | | | Merge pull request #30404 from yhirano55/fix_link_in_doc_for_active_storageRobin Dupret2017-08-251-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix link in doc for active storage [ci skip]
| * | | | | | | | | Fix link in doc for active storage [ci skip]Yoshiyuki Hirano2017-08-251-1/+1
| |/ / / / / / / /
* / / / / / / / / Should work inverse association when eager loadingRyuta Kamizono2017-08-254-15/+24
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This regression was caused by caa178c1. The block for `set_inverse_instance` should also be passed to join dependency. Fixes #30402.
* | | | | | | | Merge pull request #30401 from yhirano55/add_paths_to_docs_for_active_storageRyuta Kamizono2017-08-251-0/+1
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | Add paths to docs for Active Storage
| * | | | | | | Add paths to docs for Active StorageYoshiyuki Hirano2017-08-251-0/+1
|/ / / / / / /
* | | | | | | Merge pull request #30398 from yahonda/seed52024Ryuta Kamizono2017-08-251-0/+4
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Address `test_after_save_callback_with_autosave` failure
| * | | | | | | Address `test_after_save_callback_with_autosave` failureYasuo Honda2017-08-241-0/+4
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | when other `AutomaticInverseFindingTests` load `:comments` fixture but does not load `:posts`. Refer #30385 for similar issue
* | | | | | | Widen blob size columnGeorge Claghorn2017-08-241-1/+1
| | | | | | |
* | | | | | | Remove unused returning value `stream`Ryuta Kamizono2017-08-241-2/+0
| | | | | | |
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-20 07:04:34 +0000