Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove needless `silence_warnings` | yuuji.yaginuma | 2017-09-09 | 3 | -17/+3 |
| | | | | Since ff30db1, warning is not shown. | ||||
* | Remove unused `fork` arg for `rails` | yuuji.yaginuma | 2017-09-08 | 1 | -4/+2 |
| | |||||
* | Reorganize secrets test to use only `isolation/abstract_unit` | yuuji.yaginuma | 2017-09-07 | 1 | -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-command | Matthew Draper | 2017-09-05 | 21 | -289/+339 |
|\ | | | | | Run in-app rails commands via fork+load where possible | ||||
| * | Run in-app rails commands via fork+load where possible | Matthew Draper | 2017-09-04 | 21 | -289/+339 |
| | | | | | | | | | | While this avoids shell argument parsing, we still pass through everything in our stack. | ||||
* | | Explicitly require `test_unit/reporter` in test helper | yuuji.yaginuma | 2017-09-04 | 2 | -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 here | Matthew Draper | 2017-09-04 | 1 | -0/+2 |
| | |||||
* | Preload some additional specific files | Matthew Draper | 2017-09-04 | 1 | -0/+3 |
| | |||||
* | Preload Rails component gems in railties tests | Matthew Draper | 2017-09-04 | 4 | -2/+22 |
| | |||||
* | Run all railties tests before reporting failure | Matthew Draper | 2017-09-03 | 1 | -1/+14 |
| | |||||
* | Don't start a new process for every test file | Matthew Draper | 2017-09-03 | 1 | -10/+34 |
| | | | | | This effectively reverts 200cf32e207728df287cac2ec113a7cbe277c1eb, restoring a variant of 5a0e0e72995472e315738dcea5b5a12d6e3d3489. | ||||
* | Remove needless `silence_warnings` | yuuji.yaginuma | 2017-09-01 | 1 | -4/+1 |
| | | | | Since ff30db1, warning is not show. | ||||
* | Merge pull request #30323 from yhirano55/support_multiple_version_for_gem_method | Matthew Draper | 2017-08-31 | 3 | -9/+25 |
|\ | | | | | Support multiple versions arguments for `gem` method of Generators | ||||
| * | Support multiple versions arguments for `gem` method of Generators | Yoshiyuki Hirano | 2017-08-19 | 3 | -9/+25 |
| | | |||||
* | | Update RDOC_MAIN.rdoc [ci skip] | Yoshiyuki Hirano | 2017-08-31 | 1 | -42/+60 |
| | | |||||
* | | Fix `can't modify frozen String` error in `DBConsole` | yuuji.yaginuma | 2017-08-29 | 2 | -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 Hansen | 2017-08-28 | 3 | -5/+5 |
|\ \ | | | | | | | | | | | | | yhirano55/fix_indentation_in_secrets_yaml_template Fix indentation in the template for secrets | ||||
| * | | Fix indentation in the template for secrets | Yoshiyuki Hirano | 2017-08-23 | 3 | -5/+5 |
| | | | |||||
* | | | Use tt in doc for railties [skip ci] | Yoshiyuki Hirano | 2017-08-27 | 3 | -6/+6 |
| | | | |||||
* | | | Remove Active Storage duplicated classes from the API site | Robin Dupret | 2017-08-26 | 1 | -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 Storage | Yoshiyuki Hirano | 2017-08-25 | 1 | -0/+1 |
| | | | |||||
* | | | Merge pull request #30366 from MrOutis/action-cable-env-var | Rafael França | 2017-08-23 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | Use REDIS_URL var for Action Cable config template | ||||
| * | | | Use REDIS_URL var for Action Cable config template | Mr. Outis | 2017-08-22 | 1 | -1/+1 |
| | | | | |||||
* | | | | Merge pull request #30371 from yhirano55/fix_spaces_in_the_comment_block | Richard Schneeman | 2017-08-22 | 1 | -1/+1 |
|\ \ \ \ | |/ / / |/| | | | Fix spaces in the comment block [ci skip] | ||||
| * | | | Fix spaces in the comment block [ci skip] | Yoshiyuki Hirano | 2017-08-23 | 1 | -1/+1 |
| |/ / | |||||
* | | | Merge pull request #30348 from koic/remove_frozen_string_literal_from_templates | George Claghorn | 2017-08-21 | 42 | -85/+0 |
|\ \ \ | | | | | | | | | Remove frozen_string_literal magic comment from templates | ||||
| * | | | Remove frozen_string_literal magic comment from templates | Koichi ITO | 2017-08-22 | 42 | -85/+0 |
| | | | | |||||
* | | | | Merge pull request #30352 from yhirano55/update_licenses_link | Ryuta Kamizono | 2017-08-22 | 3 | -5/+4 |
|\ \ \ \ | |/ / / |/| | | | Update MIT licenses link [ci skip] | ||||
| * | | | Update MIT licenses link [ci skip] | Yoshiyuki Hirano | 2017-08-22 | 3 | -5/+4 |
| |/ / | |||||
* / / | Fix broken link to mysql docs in database.yml [ci skip] | Yoshiyuki Hirano | 2017-08-22 | 2 | -2/+2 |
|/ / | |||||
* | | Merge pull request #30332 from y-yagi/fix_29306 | George Claghorn | 2017-08-20 | 7 | -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 path | yuuji.yaginuma | 2017-08-21 | 7 | -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 require | yuuji.yaginuma | 2017-08-19 | 1 | -1/+0 |
|/ | | | | Since 0d72489, this file does not use `method_source`. | ||||
* | Move common tests from `AppGeneratorTest` and `PluginGeneratorTest` to ↵ | bogdanvlviv | 2017-08-17 | 5 | -200/+118 |
| | | | | `SharedGeneratorTests` | ||||
* | Merge pull request #30260 from kamipo/remove_unused_action_arg | Rafael França | 2017-08-16 | 1 | -2/+3 |
|\ | | | | | Remove unused `action` arg for `generate_routing_code` | ||||
| * | Remove unused `action` arg for `generate_routing_code` | Ryuta Kamizono | 2017-08-15 | 1 | -2/+3 |
| | | |||||
* | | Fix RuboCop offenses | Koichi ITO | 2017-08-16 | 12 | -31/+31 |
| | | | | | | | | And enable `context_dependent` of Style/BracesAroundHashParameters cop. | ||||
* | | Add test case to make sure Parameters configuration are executed once | Rafael Mendonça França | 2017-08-15 | 1 | -0/+10 |
| | | | | | | | | Test case for #30045 | ||||
* | | Add --skip-yarn option to the plugin generator | bogdanvlviv | 2017-08-15 | 7 | -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_generator | Kasper Timm Hansen | 2017-08-15 | 1 | -1/+1 |
|\ \ | | | | | | | Fix typo for api_app_generator_test | ||||
| * | | Fix typo for api_app_generator_test | Yoshiyuki Hirano | 2017-08-15 | 1 | -1/+1 |
| | | | |||||
* | | | Formatting CHANGELOGs [ci skip] | Ryuta Kamizono | 2017-08-15 | 1 | -2/+2 |
|/ / | | | | | | | | | | | Add missing backticks Add missing * Add missing . | ||||
* / | Fix format of generator docs [ci skip] | Yoshiyuki Hirano | 2017-08-15 | 1 | -1/+1 |
|/ | |||||
* | Merge pull request #30219 from yhirano55/fix_app_generator_api_options | Rafael França | 2017-08-14 | 7 | -29/+48 |
|\ | | | | | Refactor `--api` option of the app generator | ||||
| * | Refactor `--api` option of the app generator | Yoshiyuki Hirano | 2017-08-13 | 7 | -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_seconds | Rafael França | 2017-08-14 | 2 | -2/+2 |
|\ \ | | | | | | | Refactor max-age seconds | ||||
| * | | Refactor max-age seconds | Yoshiyuki Hirano | 2017-08-14 | 2 | -2/+2 |
| |/ | |||||
* | | Merge pull request #30235 from bogdanvlviv/omit-additional-removing-cable_yml | Rafael França | 2017-08-14 | 1 | -1/+0 |
|\ \ | | | | | | | Omit additional removing `config/cable.yml` | ||||
| * | | Omit additional removing `config/cable.yml` by the app generator | bogdanvlviv | 2017-08-13 | 1 | -1/+0 |
| |/ | | | | | | | `config/cable.yml` is never created if `--skip-action-cable` is given. | ||||
* | | Merge pull request #30239 from bogdanvlviv/improve-plugin_generator_code | Rafael França | 2017-08-14 | 1 | -2/+1 |
|\ \ | | | | | | | Improve plugin generator codebase |