Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| | * | | Merge pull request #27042 from kirs/yaml-schema-cache | Rafael França | 2016-12-13 | 1 | -2/+2 | |
| | |\ \ | | | | | | | | | | | Schema cache in YAML | |||||
| | | * | | Use YAML to serialize schema cache | Kir Shatrov | 2016-11-27 | 1 | -2/+2 | |
| | | | | | ||||||
| | * | | | Don't assign a default to `webpack`. | Kasper Timm Hansen | 2016-12-12 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unintentionally makes `--webpack` implied on `rails new apper`. If passed `--webpack` Thor assigns `"webpack"` to `options[:webpack]`, so we can check for that instead of `"base"`. | |||||
| | * | | | Add option to trigger JS framework-specific installation as part of webpack ↵ | David Heinemeier Hansson | 2016-12-12 | 2 | -2/+17 | |
| | | | | | | | | | | | | | | | | | | | | setup | |||||
| | * | | | Oops! | Akira Matsuda | 2016-12-12 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | namespace_ladder can be nil here | |||||
| | * | | | :nail_care: | Akira Matsuda | 2016-12-12 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | | | | | | | Don't expect the caller of this method to know that the return value has an extra " " | |||||
| | * | | | run `webpack` command only when webpack option is specified | yuuji.yaginuma | 2016-12-10 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | gemfile entry method need to return an empty array rather than nil (#27318) | Yuji Yaginuma | 2016-12-09 | 1 | -4/+4 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes the following error when executing rails new command. ``` (erb):9:in `block in template': undefined method `comment' for nil:NilClass (NoMethodError) ``` Follow up to #27288 | |||||
| | * | | | Use GitHub webpacker until closer to release | David Heinemeier Hansson | 2016-12-09 | 1 | -1/+1 | |
| | | | | | ||||||
| | * | | | Basic --webpack delegation to new webpacker gem (#27288) | David Heinemeier Hansson | 2016-12-09 | 2 | -1/+16 | |
| | | | | | ||||||
| | * | | | Show message if Yarn is not installed (#27312) | Guillermo Iguaran | 2016-12-09 | 1 | -1/+8 | |
| | | | | | ||||||
| | * | | | Only have Yarn bundling commented out as we cant be sure Yarn is installed | David Heinemeier Hansson | 2016-12-08 | 2 | -5/+2 | |
| | | | | | ||||||
| | * | | | Fix ability to run railties test suite | David Heinemeier Hansson | 2016-12-08 | 1 | -0/+1 | |
| | | | | | ||||||
| | * | | | fix broken tests (#27308) | Yuji Yaginuma | 2016-12-08 | 1 | -3/+2 | |
| | | | | | | | | | | | | | | | Follow up to 0d20530e5edfd7d00fbc2a38ef5f87eca6ccc924 | |||||
| | * | | | specify `skip_yarn` option in API-only Application (#27309) | Yuji Yaginuma | 2016-12-08 | 2 | -3/+12 | |
| | | | | | ||||||
| | * | | | Merge pull request #25051 from bf4/extract_notes_as_binary | Sean Griffin | 2016-12-08 | 1 | -1/+1 | |
| | |\ \ \ | | | | | | | | | | | | | Extract notes as binary | |||||
| | | * | | | Extract notes from files in binary | Benjamin Fleischer | 2016-09-05 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prevents: ArgumentError: invalid byte sequence in UTF-8 railties/lib/rails/source_annotation_extractor.rb:115:in `=~' railties/lib/rails/source_annotation_extractor.rb:115:in `block in extract_annotations_from' And there's no reason we need to interpret the files as UTF-8 when scanning for annotations. Applies to Rails 4.2 as well. | |||||
| | * | | | | Make Yarn the default, drop default vendor/asset directories (#27300) | David Heinemeier Hansson | 2016-12-08 | 11 | -111/+34 | |
| | | | | | | ||||||
| | * | | | | Update puma.rb | brchristian | 2016-12-05 | 1 | -6/+6 | |
| | | | | | | | | | | | | | | | | | | Clarifying some of the language/punctuation and removing a couple comma splices. | |||||
| | * | | | | Add fullstop following the pattern used in rest of the file | Prathamesh Sonpatki | 2016-12-05 | 1 | -1/+1 | |
| | | | | | | ||||||
| | * | | | | Fix binstop -> binstub typo in railties changelog. | Josef Šimánek | 2016-12-05 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | [ci skip] | |||||
| | * | | | | Use JavaScripts instead of Javascripts in `rake stats` | Koichi ITO | 2016-12-03 | 1 | -1/+1 | |
| | | | | | | ||||||
| | * | | | | use yarn binstub in `bin/update` | yuuji.yaginuma | 2016-12-02 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | Follow up to 3dac36bd8e26363bb10f4d2a7b21efa75d200e26 | |||||
| * | | | | | clear all environments log files by default | yuuji.yaginuma | 2016-10-14 | 3 | -2/+43 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In #22703, `log:clear` task has been changed to clear only standard environment log files. However, it is often to add a non-standard environment(e.g. "staging"). Therefore, I think than it is better to clear all environments log files by default. | |||||
* | | | | | | use Thor option parser in server commands parse | yuuji.yaginuma | 2016-12-24 | 2 | -90/+110 | |
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `ServerCommand` inherits Thor, but currently does not use Thor option parser. Therefore, if leave the argument of Thor as it is, it becomes an error by the argument checking of Thor. To avoid it, to use the Thor option parser instead of reimplementing it. Fixes #26964 | |||||
* | | | | | Yarn: Move node_modules, package.json, and yarn.lock file to vendor (#27245) | David Heinemeier Hansson | 2016-12-01 | 8 | -12/+16 | |
| | | | | | | | | | | | | | | | Move node_modules, package.json, and yarn.lock file to vendor | |||||
* | | | | | use `Gem.win_platform?` to check windows Ruby platforms | yuuji.yaginuma | 2016-11-30 | 4 | -5/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | `Gem.win_platform?` check if it is Windows more accurately. Ref: https://github.com/ruby/ruby/blob/ruby_2_2/lib/rubygems.rb#L945..L952 | |||||
* | | | | | Merge pull request #27221 from phoet/current_is_not_used | Matthew Draper | 2016-11-30 | 1 | -1/+0 | |
|\ \ \ \ \ | | | | | | | | | | | | | removed `@current` as it is not used | |||||
| * | | | | | removed `@current` as it is not used | phoet | 2016-11-30 | 1 | -1/+0 | |
| | | | | | | ||||||
* | | | | | | fix typo in changelog | Robert Glaser | 2016-11-30 | 1 | -1/+1 | |
|/ / / / / | ||||||
* | | | | | Let's use only vendored rails-ujs while we start to publish it to npm registry | Guillermo Iguaran | 2016-11-29 | 1 | -3/+1 | |
| | | | | | ||||||
* | | | | | Add CHANGELOG entry for --yarn option | Liceth Ovalles | 2016-11-29 | 1 | -0/+5 | |
| | | | | | ||||||
* | | | | | Merge pull request #26836 from Liceth/npm | Guillermo Iguaran | 2016-11-29 | 8 | -3/+106 | |
|\ \ \ \ \ | | | | | | | | | | | | | Add Yarn support in new apps using --yarn option | |||||
| * | | | | | Run yarn on setup and update scripts. | Liceth Ovalles | 2016-11-28 | 2 | -2/+8 | |
| | | | | | | ||||||
| * | | | | | Add node_modules to default gitignore | Liceth Ovalles | 2016-11-28 | 1 | -0/+5 | |
| | | | | | | ||||||
| * | | | | | Add node_modules path to assets load paths when --yarn option is used | Liceth Ovalles | 2016-11-28 | 2 | -0/+5 | |
| | | | | | | ||||||
| * | | | | | Add package.json for Yarn if --yarn option is added | Liceth Ovalles | 2016-11-28 | 4 | -1/+88 | |
| | | | | | | ||||||
* | | | | | | rails-ujs is now shipped with Action View | Guillermo Iguaran | 2016-11-29 | 4 | -8/+2 | |
| | | | | | | ||||||
* | | | | | | use correct variable in BacktraceCleaner test | yuuji.yaginuma | 2016-11-26 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | `@target_dir` variable was changed to local variable in 8e1714b. | |||||
* | | | | | | Merge pull request #27140 from ↵ | Rafael França | 2016-11-25 | 2 | -1/+19 | |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | supercaracal/fix-force-ssl-if-session-store-disabled Fix a force ssl redirection bug that occur when session store disabled. | |||||
| * | | | | | | Remove aggressive unit test with mock. And add integration level test. | Taishi Kasuga | 2016-11-24 | 2 | -1/+19 | |
| | | | | | | | ||||||
* | | | | | | | [ci skip] `MiniTest` -> `Minitest` | Fumiaki MATSUSHIMA | 2016-11-24 | 1 | -1/+1 | |
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | MiniTest was renamed to Minitest. Already renamed on https://github.com/rails/rails/pull/13366 But slipped into on https://github.com/rails/rails/pull/18413/files#diff-6bb90a693835b0e92910b796c8b0ef59R27 | |||||
* | | | | | | Add CHANGELOG entry | Guillermo Iguaran | 2016-11-21 | 1 | -0/+5 | |
| | | | | | | ||||||
* | | | | | | Don't add jQuery by default in new apps and use rails-ujs as UJS adapter instead | Guillermo Iguaran | 2016-11-21 | 5 | -18/+20 | |
|/ / / / / | ||||||
* | | / / | deep symbolize keys on secrets.yml | Isaac Sloan | 2016-11-21 | 3 | -2/+22 | |
| |_|/ / |/| | | | ||||||
* | | | | Pass `rails runner` args onto file again. | Kasper Timm Hansen | 2016-11-20 | 2 | -1/+10 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When making the new command insfrastructure I had missed that `bin/rails runner some_file.rb some args` would pass the extra args onto the file in `ARGV`. Now fixed by allowing the command to take extra args again, and make sure to remove the file name from `ARGV`. | |||||
* | | | | Use existing --skip-turbolinks option for conditionals instead of checking ↵ | Guillermo Iguaran | 2016-11-19 | 2 | -2/+2 | |
| | | | | | | | | | | | | | | | | if turbolinks is present in gemfile list | |||||
* | | | | Merge pull request #27106 from prathamesh-sonpatki/add-missing-test | Eileen M. Uchitelle | 2016-11-19 | 1 | -0/+20 | |
|\ \ \ \ | | | | | | | | | | | Add missing test for singular resource output in rake routes | |||||
| * | | | | Add missing test for singular resource output in rake routes | Prathamesh Sonpatki | 2016-11-19 | 1 | -0/+20 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - This test was present in https://github.com/rails/rails/pull/27089 but not present on master, may be removed in merge commit? - There was discussion about moving this to `application/rake_test` so may be this happened in merge commit. - https://github.com/rails/rails/pull/27089#discussion_r88731157 | |||||
* | | | | | move CHANGELOG entry to the appropriate file [ci skip] | yuuji.yaginuma | 2016-11-19 | 1 | -6/+0 | |
|/ / / / | | | | | | | | | | | | | Follow up to #27098 |