Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Prevent race condition when launching EventMachine reactor | Matthew Draper | 2016-11-30 | 2 | -2/+28 |
| | | | | | | | reactor_running? will be true just after the thread enters EventMachine.run; reactor_thread only gets set after the internal initialize_event_machine method has been called, the C extension is set up, and it is entering its run loop. | ||||
* | Avoid race condition in AJ integration tests | Matthew Draper | 2016-11-30 | 2 | -1/+3 |
| | | | | Make sure the file doesn't exist until we've finished writing it. | ||||
* | 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 |
| | | |||||
* | | Merge pull request #27220 from mrreynolds/patch-1 | Kasper Timm Hansen | 2016-11-30 | 1 | -1/+1 |
|\ \ | |/ |/| | fix typo in changelog | ||||
| * | fix typo in changelog | Robert Glaser | 2016-11-30 | 1 | -1/+1 |
|/ | |||||
* | Missing require 'active_support/notifications' | Akira Matsuda | 2016-11-30 | 1 | -0/+1 |
| | |||||
* | Merge pull request #27217 from roryokane/patch-1 | Rafael França | 2016-11-29 | 1 | -1/+1 |
|\ | | | | | Fix typo in Rails 5.0 release notes – “when when” | ||||
| * | Fix typo in Rails 5.0 release notes – “when when” | Rory O’Kane | 2016-11-29 | 1 | -1/+1 |
|/ | | | [ci skip] | ||||
* | Merge pull request #27212 from dcarral/docs-action-on-unpermitted-parameters ↵ | Sean Griffin | 2016-11-29 | 1 | -2/+2 |
|\ | | | | | | | | | [ci skip] Explicitly document 'false' as value for 'action_on_unpermitted_parameters' | ||||
| * | Document 'false' as option for 'action_on_unpermitted_parameters' | Daniel Carral | 2016-11-29 | 1 | -2/+2 |
| | | | | | | | | [ci skip] | ||||
* | | Treat combined durations as a single unit | Sean Griffin | 2016-11-29 | 2 | -3/+20 |
|/ | | | | | | | | | | | | Prior to this commit, `3.months - 3.months` would result in a duration that has the "parts" of `[[:months, 3], [:months, -3]]`. This would mean that it was subtly different than `2.months - 2.months`. When applied to a time, the date might actually change if the resulting day doesn't exist however many months in the future, even though in both cases we just wanted to add `0`, which should always be an identity operation. With this change, we now store the parts as a hash, so `3.months - 3.months` is simply stored as `{ months: 0 }`. | ||||
* | Merge pull request #27126 from kamipo/fix_unsigned_with_zerofill | Rafael França | 2016-11-29 | 4 | -2/+9 |
|\ | | | | | Fix that unsigned with zerofill is treated as signed | ||||
| * | Fix that unsigned with zerofill is treated as signed | Ryuta Kamizono | 2016-11-27 | 4 | -2/+9 |
| | | | | | | | | Fixes #27125. | ||||
* | | 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 | 6 | -17/+2 |
| | | | |||||
* | | | Merge pull request #27204 from JessRudder/22862-auth-note-on-tests | Jon Moss | 2016-11-28 | 1 | -0/+7 |
|\ \ \ | | | | | | | | | Add note about step needed to get tests to pass if you implemented basic auth [ci skip] | ||||
| * | | | Add note about step needed to get tests to pass if you implemented basic ↵ | JessRudder | 2016-11-28 | 1 | -0/+7 |
|/ / / | | | | | | | | | | auth [ci skip] | ||||
* | | | Merge pull request #27196 from kirs/rubocop-gem | Rafael França | 2016-11-28 | 2 | -0/+16 |
|\ \ \ | | | | | | | | | Add rubocop to Gemfile | ||||
| * | | | Add rubocop to Gemfile | Kir Shatrov | 2016-11-28 | 2 | -0/+16 |
| | | | | |||||
* | | | | Merge pull request #27195 from y-yagi/use_public_module_include | Santiago Pastorino | 2016-11-27 | 3 | -3/+3 |
|\ \ \ \ | | | | | | | | | | | use public Module#include instead of send :include | ||||
| * | | | | use public Module#include instead of send :include | yuuji.yaginuma | 2016-11-27 | 3 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | Follow up to #18767 | ||||
* | | | | | Merge pull request #27190 from jnoortheen/issue-27189 | Santiago Pastorino | 2016-11-27 | 1 | -3/+4 |
|\ \ \ \ \ | | | | | | | | | | | | | I found this issue and wanted to give the fix. | ||||
| * | | | | | fix: solves issue#27189 | jnoortheen | 2016-11-27 | 1 | -3/+4 |
| | | | | | | |||||
* | | | | | | Merge pull request #27193 from plentz/master | Santiago Pastorino | 2016-11-27 | 2 | -2/+28 |
|\ \ \ \ \ \ | | | | | | | | | | | | | | | fixing update_all and delete_all when chained with left_joins | ||||
| * | | | | | | fixing update_all and delete_all when chained with left_joins. fixes #27192 | Diego Plentz | 2016-11-27 | 2 | -2/+28 |
|/ / / / / / | |||||
* / / / / / | [ci skip] Update the `as` docs. | Kasper Timm Hansen | 2016-11-27 | 1 | -1/+2 |
|/ / / / / | | | | | | | | | | | | | | | | Mention the Accept header and how that figures into the request format. | ||||
* | | | / | Remove default argument value. | Kasper Timm Hansen | 2016-11-27 | 1 | -1/+1 |
| |_|_|/ |/| | | | | | | | | | | | Wrongly added when fixing the request path wrangling. | ||||
* | | | | Merge pull request #26865 from ↵ | Matthew Draper | 2016-11-27 | 3 | -4/+23 |
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | sergey-alekseev/uniqueness-validation-scope-with-polymorphic-association fix the uniqueness validation scope with a polymorphic association | ||||
| * | | | | fix the uniqueness validation scope with a polymorphic association | Sergey Alekseev | 2016-11-26 | 3 | -4/+23 |
| | | | | | | | | | | | | | | | | | | | | https://gist.github.com/sergey-alekseev/946657ebdb5e58d1bee115714056ec96 | ||||
* | | | | | abort_on_exception can't be disabled for a single thread | Matthew Draper | 2016-11-27 | 2 | -6/+8 |
| | | | | | | | | | | | | | | | | | | | | | | | | | If it's enabled globally, it's on regardless of how individual threads are set. | ||||
* | | | | | Only start rabbitmq when we need it | Matthew Draper | 2016-11-27 | 1 | -6/+19 |
| | | | | | |||||
* | | | | | We expect exceptions; abort will just raise in the wrong place | Matthew Draper | 2016-11-27 | 2 | -9/+17 |
| | | | | | |||||
* | | | | | Clear available connections immediately | Matthew Draper | 2016-11-27 | 1 | -0/+2 |
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It'll be re-cleared when it's rebuilt in with_new_connections_blocked's ensure, but we still need to clear it inside this synchronize -- we've disconnected connections that may be available in the queue, and while other threads are not allowed to make *new* connections, they are still allowed to take existing ones from there. This was incorrectly removed in d314646c965b045724e6bdb9d61dcecfabc0ba8f. | ||||
* | | | | Merge pull request #27183 from maclover7/jm-av-release | Guillermo Iguaran | 2016-11-26 | 1 | -5/+7 |
|\ \ \ \ | |/ / / |/| | | | Update RELEASING_RAILS.md for rails-ujs | ||||
| * | | | Update RELEASING_RAILS.md for rails-ujs | Jon Moss | 2016-11-26 | 1 | -5/+7 |
|/ / / | | | | | | | | | | [ci skip] | ||||
* | | | Merge pull request #27182 from maclover7/jm-actionview-build | Guillermo Iguaran | 2016-11-26 | 1 | -1/+9 |
|\ \ \ | | | | | | | | | Add asset compilation Rake task for UJS | ||||
| * | | | Add asset compilation Rake task for UJS | Jon Moss | 2016-11-26 | 1 | -1/+9 |
|/ / / | | | | | | | | | | | | | Uses same build pipeline (Blade) as Action Cable. Will run when Rails gems are being built for release. | ||||
* | | | Add UJS tests | Guillermo Iguaran | 2016-11-26 | 22 | -0/+5280 |
| | | | |||||
* | | | Add rails-ujs to Action View | Guillermo Iguaran | 2016-11-26 | 14 | -0/+649 |
| | | | |||||
* | | | Convert ActionView::Railtie into Engine | Guillermo Iguaran | 2016-11-26 | 1 | -1/+1 |
| | | | |||||
* | | | Make the test that seems to be getting stuck noisier | Matthew Draper | 2016-11-26 | 1 | -35/+55 |
| | | | | | | | | | | | | | | | I assume it's upset because of the change in d314646c965b045724e6bdb9d61dcecfabc0ba8f, but I don't yet understand why. | ||||
* | | | Merge pull request #27179 from y-yagi/follow_up_to_8e1714b | Matthew Draper | 2016-11-26 | 1 | -1/+1 |
|\ \ \ | | | | | | | | | use correct variable in BacktraceCleaner test | ||||
| * | | | use correct variable in BacktraceCleaner test | yuuji.yaginuma | 2016-11-26 | 1 | -1/+1 |
|/ / / | | | | | | | | | | `@target_dir` variable was changed to local variable in 8e1714b. |