| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Follow up of #24844.
The key of `@owner_to_pool` was changed from `klass.name` to
`spec.name`. By this change "memory leaks in development mode"
will not happen, bacause the equality of string is not changed
by reloading of model files.
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
Getting Started Guide: Update Ruby version to 2.3.1p112 [ci skip]
|
|/ / / /
| | | |
| | | |
| | | | |
[ci skip]
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
remove_connection
When calling `remove_connection` on a model, we delete the pool so we also
need to reset the `connection_specification_name` so it will fallback to
the parent.
This was the current behavior before rails 5, which will fallback to the
parent connection pool.
[fixes #24959]
Special thanks to @jrafanie for working with me on this fix.
|
|\ \ \ \
| | | | |
| | | | | |
Minor syntax change
|
|/ / / /
| | | |
| | | | |
While working on this very example recently, I found that Rails will throw POST errors if you leave an empty block in front the `format.js` when using either `remote: true` or vanilla JS/JQuery.
|
|\ \ \ \
| | | | |
| | | | | |
Should quote `lock_name` to pass to `get_advisory_lock`
|
| | | | | |
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
Replacement cycle for readability
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Fix Gemfile.lock by updating rails-dom-testing
|
|/ / / / /
| | | | |
| | | | |
| | | | | |
- Followup of https://github.com/rails/rails/pull/24946.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Update rails-dom-testing gem to 2.0
|
|/ / / / /
| | | | |
| | | | |
| | | | | |
Resolves #24924.
|
|\ \ \ \ \
| | | | | |
| | | | | | |
use Rack::Utils.valid_path? to check path
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
This commit uses the new method in Rack to check if a path is valid.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Improve AV changelog with regards to default RAW handler
|
| |/ / / / /
| | | | | |
| | | | | |
| | | | | | |
It highlights which kind of incompatibilities this could lead to when rendering plain JS or HTML partials.
|
|\ \ \ \ \ \
| | | | | | |
| | | | | | | |
Feature getting started guide [ci skip]
|
|/ / / / / / |
|
|\ \ \ \ \ \
| |_|/ / / /
|/| | | | | |
Add :ssl/:tls to ActionMailer rdoc [ci skip]
|
| | | | | | |
|
|\ \ \ \ \ \
| |/ / / / /
|/| | | | | |
WIP: Action Cable JavaScript tests
|
| |/ / / / |
|
| | | | |
| | | | |
| | | | |
| | | | | |
Will be using this in later code.
|
|/ / / /
| | | |
| | | |
| | | |
| | | | |
Each test executes in another tmp folder, so there's no chance
of polluting another test run.
|
|\ \ \ \
| | | | |
| | | | | |
Replacement cycle for readability
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | | |
Change http github links to https [ci skip]
|
| | | | |
| | | | |
| | | | |
| | | | | |
Follow up to 7d55c846f11a435a4c7aae58f99279a2b9dd8506
|
|\ \ \ \ \
| | | | | |
| | | | | | |
Use consistent term `underscored`
|
|/ / / / / |
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Use original `Array#sum` when calculating Numeric sum.
This commit is related #24804 issue.
Issue #24804 reports `Array#sum` becomes much slower when
ActiveSupport is included.
This commit tries to use original method as far as possible.
```shell
$ cat array_sum.rb
class Array
alias core_sum sum
end
require 'benchmark/ips'
require 'active_support/core_ext/enumerable'
ary = [1.0] * 1_000_000
Benchmark.ips do |x|
x.report("core sum") { ary.core_sum }
x.report("AS's sum") { ary.sum }
x.compare!
end
$ bundle exec ruby -v -I lib array_sum.rb
ruby 2.4.0dev (2016-05-01 master 54867) [x86_64-darwin14]
Calculating -------------------------------------
core sum 4.000 i/100ms
AS's sum 5.000 i/100ms
-------------------------------------------------
core sum 50.492 (± 7.9%) i/s - 252.000
AS's sum 50.116 (± 6.0%) i/s - 250.000
Comparison:
core sum: 50.5 i/s
AS's sum: 50.1 i/s - 1.01x slower
```
Signed-off-by: Jeremy Daer <jeremydaer@gmail.com>
|
|\ \ \ \
| | | | |
| | | | | |
Followup to #24844
|
|/ / / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Some slight documentation edits and fixes. Also, run remove unnecessary
`RuntimeError`.
r? @arthurnn
|
| | | |
| | | |
| | | |
| | | |
| | | | |
In #23935, cable file was to be provided by the javascript instead of coffeescript,
doc was also been modified to use javascript.
|
|\ \ \ \
| | | | |
| | | | | |
Replace `loop` to `until`
|
|/ / / / |
|
|\ \ \ \
| | | | |
| | | | |
| | | | | |
Preparing for 5.0.0.rc1 release
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | | |
http --> https
|
| |/ / / /
| | | | |
| | | | |
| | | | | |
[ci skip]
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
Helpers doc grammar fix [skip ci]
|
|/ / / / |
|
| | | |
| | | |
| | | |
| | | | |
It is related with https://github.com/rack/rack/commit/7b66d2cdb80a4d6b44fa8c61d92e25fbbda1f152
|