aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
...
* | Remove needless `silence_warnings`yuuji.yaginuma2017-09-093-17/+3
| | | | | | | | Since ff30db1, warning is not shown.
* | Remove unused `fork` arg for `rails`yuuji.yaginuma2017-09-081-4/+2
| |
* | Reorganize secrets test to use only `isolation/abstract_unit`yuuji.yaginuma2017-09-071-14/+18
| | | | | | | | | | | | | | | | Currently, secrets test uses `abstract_unit` and `isolation/abstract_unit`. This is a bit odd. Therefore, reorganize it so that use only `isolation/abstract_unit`. Context: https://github.com/rails/rails/pull/30520#issuecomment-327409586
* | Merge pull request #30520 from rails/railties-rails-commandMatthew Draper2017-09-0521-289/+339
|\ \ | | | | | | Run in-app rails commands via fork+load where possible
| * | Run in-app rails commands via fork+load where possibleMatthew Draper2017-09-0421-289/+339
| | | | | | | | | | | | | | | While this avoids shell argument parsing, we still pass through everything in our stack.
* | | Explicitly require `test_unit/reporter` in test helperyuuji.yaginuma2017-09-042-0/+7
|/ / | | | | | | | | | | | | | | | | | | | | If the user used the `bin/test` to execute the test, this file is automatically loaded, so require is unnecessary. https://github.com/rails/rails/blob/acea68de026ba657cb65c4dd0fc1f24ba67e1cf8/railties/lib/rails/plugin/test.rb#L4 However, when using `rake test`, an explicit require is required because the above file is not loaded. Fixes #30516
* | Don't need the layout hereMatthew Draper2017-09-041-0/+2
| |
* | Preload some additional specific filesMatthew Draper2017-09-041-0/+3
| |
* | Preload Rails component gems in railties testsMatthew Draper2017-09-044-2/+22
| |
* | Run all railties tests before reporting failureMatthew Draper2017-09-031-1/+14
| |
* | Don't start a new process for every test fileMatthew Draper2017-09-031-10/+34
| | | | | | | | | | This effectively reverts 200cf32e207728df287cac2ec113a7cbe277c1eb, restoring a variant of 5a0e0e72995472e315738dcea5b5a12d6e3d3489.
* | Remove needless `silence_warnings`yuuji.yaginuma2017-09-011-4/+1
| | | | | | | | Since ff30db1, warning is not show.
* | Merge pull request #30323 from yhirano55/support_multiple_version_for_gem_methodMatthew Draper2017-08-313-9/+25
|\ \ | | | | | | Support multiple versions arguments for `gem` method of Generators
| * | Support multiple versions arguments for `gem` method of GeneratorsYoshiyuki Hirano2017-08-193-9/+25
| | |
* | | Update RDOC_MAIN.rdoc [ci skip]Yoshiyuki Hirano2017-08-311-42/+60
| | |
* | | Fix `can't modify frozen String` error in `DBConsole`yuuji.yaginuma2017-08-292-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | Without this, `dbconsole` raises an error as follwing: ``` RuntimeError: can't modify frozen String railties/lib/rails/commands/dbconsole/dbconsole_command.rb:79:in `start' ```
* | | Merge pull request #30370 from ↵Kasper Timm Hansen2017-08-283-5/+5
|\ \ \ | | | | | | | | | | | | | | | | yhirano55/fix_indentation_in_secrets_yaml_template Fix indentation in the template for secrets
| * | | Fix indentation in the template for secretsYoshiyuki Hirano2017-08-233-5/+5
| | | |
* | | | Use tt in doc for railties [skip ci]Yoshiyuki Hirano2017-08-273-6/+6
| | | |
* | | | 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]
* | | | Add paths to docs for Active StorageYoshiyuki Hirano2017-08-251-0/+1
| | | |
* | | | Merge pull request #30366 from MrOutis/action-cable-env-varRafael França2017-08-231-1/+1
|\ \ \ \ | | | | | | | | | | Use REDIS_URL var for Action Cable config template
| * | | | Use REDIS_URL var for Action Cable config templateMr. Outis2017-08-221-1/+1
| | | | |
* | | | | Merge pull request #30371 from yhirano55/fix_spaces_in_the_comment_blockRichard Schneeman2017-08-221-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | Fix spaces in the comment block [ci skip]
| * | | | Fix spaces in the comment block [ci skip]Yoshiyuki Hirano2017-08-231-1/+1
| |/ / /
* | | | Merge pull request #30348 from koic/remove_frozen_string_literal_from_templatesGeorge Claghorn2017-08-2142-85/+0
|\ \ \ \ | | | | | | | | | | Remove frozen_string_literal magic comment from templates
| * | | | Remove frozen_string_literal magic comment from templatesKoichi ITO2017-08-2242-85/+0
| | | | |
* | | | | Merge pull request #30352 from yhirano55/update_licenses_linkRyuta Kamizono2017-08-223-5/+4
|\ \ \ \ \ | |/ / / / |/| | | | Update MIT licenses link [ci skip]
| * | | | Update MIT licenses link [ci skip]Yoshiyuki Hirano2017-08-223-5/+4
| |/ / /
* / / / Fix broken link to mysql docs in database.yml [ci skip]Yoshiyuki Hirano2017-08-222-2/+2
|/ / /
* | | Merge pull request #30332 from y-yagi/fix_29306George Claghorn2017-08-207-17/+44
|\ \ \ | | | | | | | | Make `restart` and `dev:cache` tasks work when customizing pid file path
| * | | Make `restart` and `dev:cache` tasks work when customizing pid file pathyuuji.yaginuma2017-08-217-17/+44
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | Originally, it hard-coded pid file path. It can not be removed when customizing pid file path. But rake task can not get pid file path. Therefore, do not remove file in rake task, makes it possible to judge whether it is restart from the argument of the command and removes the file in server command. Fixes #29306
* / / Remove unused requireyuuji.yaginuma2017-08-191-1/+0
|/ / | | | | | | Since 0d72489, this file does not use `method_source`.
* | Move common tests from `AppGeneratorTest` and `PluginGeneratorTest` to ↵bogdanvlviv2017-08-175-200/+118
| | | | | | | | `SharedGeneratorTests`
* | Merge pull request #30260 from kamipo/remove_unused_action_argRafael França2017-08-161-2/+3
|\ \ | | | | | | Remove unused `action` arg for `generate_routing_code`
| * | Remove unused `action` arg for `generate_routing_code`Ryuta Kamizono2017-08-151-2/+3
| | |
* | | Fix RuboCop offensesKoichi ITO2017-08-1612-31/+31
| | | | | | | | | | | | And enable `context_dependent` of Style/BracesAroundHashParameters cop.
* | | Add test case to make sure Parameters configuration are executed onceRafael Mendonça França2017-08-151-0/+10
| | | | | | | | | | | | Test case for #30045
* | | Add --skip-yarn option to the plugin generatorbogdanvlviv2017-08-157-28/+43
| | | | | | | | | | | | | | | Add SharedGeneratorTests#application_path This method will help to DRY in files app_generator_test.rb, plugin_generator_test.rb
* | | Merge pull request #30265 from yhirano55/fix_api_app_generatorKasper Timm Hansen2017-08-151-1/+1
|\ \ \ | | | | | | | | Fix typo for api_app_generator_test
| * | | Fix typo for api_app_generator_testYoshiyuki Hirano2017-08-151-1/+1
| | | |
* | | | Formatting CHANGELOGs [ci skip]Ryuta Kamizono2017-08-151-2/+2
|/ / / | | | | | | | | | | | | | | | Add missing backticks Add missing * Add missing .
* / / Fix format of generator docs [ci skip]Yoshiyuki Hirano2017-08-151-1/+1
|/ /
* | Merge pull request #30219 from yhirano55/fix_app_generator_api_optionsRafael França2017-08-147-29/+48
|\ \ | | | | | | Refactor `--api` option of the app generator
| * | Refactor `--api` option of the app generatorYoshiyuki Hirano2017-08-137-29/+48
| | | | | | | | | | | | | | | It can't use JavaScript in api mode. Action cable should be disabled by default.
* | | Merge pull request #30244 from yhirano55/refactor_max_age_secondsRafael França2017-08-142-2/+2
|\ \ \ | | | | | | | | Refactor max-age seconds
| * | | Refactor max-age secondsYoshiyuki Hirano2017-08-142-2/+2
| |/ /
* | | Merge pull request #30235 from bogdanvlviv/omit-additional-removing-cable_ymlRafael França2017-08-141-1/+0
|\ \ \ | | | | | | | | Omit additional removing `config/cable.yml`
| * | | Omit additional removing `config/cable.yml` by the app generatorbogdanvlviv2017-08-131-1/+0
| |/ / | | | | | | | | | `config/cable.yml` is never created if `--skip-action-cable` is given.
* | | Merge pull request #30239 from bogdanvlviv/improve-plugin_generator_codeRafael França2017-08-141-2/+1
|\ \ \ | | | | | | | | Improve plugin generator codebase