Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge pull request #21250 from ronakjangir47/safe_const | Yves Senn | 2015-09-07 | 1 | -0/+4 |
|\ | | | | | | | safe_constantize - Added Object scoped missing test cases | ||||
| * | safe_constantize - Added Object scoped missing test cases | Ronak Jangir | 2015-09-06 | 1 | -0/+2 |
| | | |||||
* | | Removed Extra ‘the’ [ci skip] | Ronak Jangir | 2015-09-06 | 1 | -1/+1 |
|/ | |||||
* | Update Unicode Version to 8.0.0 | Anshul Sharma | 2015-09-04 | 3 | -1/+5 |
| | |||||
* | docs, make `blank?` behavior clear. Closes #21468. [ci skip] | Yves Senn | 2015-09-02 | 1 | -2/+2 |
| | |||||
* | Merge pull request #21467 from yui-knk/doc/subscribe | Zachary Scott | 2015-09-02 | 1 | -2/+2 |
|\ | | | | | [ci skip] Add description about which object | ||||
| * | [ci skip] Add description about which object | yui-knk | 2015-09-02 | 1 | -2/+2 |
| | | | | | | | | `ActiveSupport::Notifications.subscribe` expects as second parameter. | ||||
* | | No need CHANGELOG entry for #21421 [ci skip] | Rafael Mendonça França | 2015-09-01 | 1 | -6/+0 |
| | | |||||
* | | ArrayInquirer to correctly find symbols or strings | Leigh Halliday | 2015-08-28 | 3 | -11/+22 |
|/ | | | | | | | | | The problem existed where if your ArrayInquirer values were strings but you checked them using any? with a symbol, it would not find the value. Now it will correctly check whether both the String form or the Symbol form are included in the Array. ` | ||||
* | Tiny documentation improvements [ci skip] | Robin Dupret | 2015-08-28 | 1 | -5/+6 |
| | |||||
* | - Extracted `DELIMITED_REGEX` to `delimited_regex` method and made use of ↵ | Vipul A M | 2015-08-28 | 4 | -2/+23 |
| | | | | | | | | | | user passed `options[:delimited_regex]` if available. Changed `DELIMITED_REGEX` to `DEFAULT)DELIMITED_REGEX` to signify what it means. - Added tests for number to delimited and number to currency in both actionview and activesupport. Changes Changes | ||||
* | 10X speed improvements for AS::Dependencies.loadable_constants_for_path | Jean Boussier | 2015-08-27 | 1 | -5/+5 |
| | | | | | | | | | | | | | | | | | | | | When the autoload_paths start to grows, this methods is quite a hotspot >> ActiveSupport::Dependencies.autoload_paths.size => 49 >> Benchmark.ips { |x| x.report('baseline') { ActiveSupport::Dependencies.loadable_constants_for_path(File.expand_path('app/models/shop')) }} Calculating ------------------------------------- baseline 90.000 i/100ms ------------------------------------------------- baseline 1.073k (±20.2%) i/s - 4.950k After the patch Calculating ------------------------------------- patched 883.000 i/100ms ------------------------------------------------- patched 11.050k (±19.7%) i/s - 50.331k | ||||
* | Merge pull request #21387 from ronakjangir47/remove_dup_minitest | Kasper Timm Hansen | 2015-08-26 | 1 | -2/+0 |
|\ | | | | | Removed duplicate require 'minitest/mock' | ||||
| * | Removed duplicate requiring minitest/mock as it is already required in ↵ | Ronak Jangir | 2015-08-26 | 1 | -2/+0 |
| | | | | | | | | method_call_assertions | ||||
* | | Fixed to_datetime docs [ci skip] | Ronak Jangir | 2015-08-26 | 1 | -2/+3 |
|/ | |||||
* | Merge pull request #21257 from jdantonio/async-job | Kasper Timm Hansen | 2015-08-25 | 1 | -1/+1 |
|\ | | | | | Initial implementation of ActiveJob AsyncAdapter. | ||||
| * | Initial implementation of ActiveJob AsyncAdapter. | Jerry D'Antonio | 2015-08-25 | 1 | -1/+1 |
| | | |||||
* | | Merge pull request #21281 from ronakjangir47/added_docs | Rafael Mendonça França | 2015-08-25 | 1 | -1/+10 |
|\ \ | |/ |/| | Added docs for TimeWithZone [ci skip] | ||||
| * | Added docs for TimeWithZone [ci skip] | Ronak Jangir | 2015-08-18 | 1 | -1/+10 |
| | | |||||
* | | use `caller_locations` instead of `caller` | Aaron Patterson | 2015-08-24 | 2 | -3/+14 |
| | | | | | | | | | | We have `caller_locations`, so we don't need to parse the strings in the callstack. | ||||
* | | minor documentation improvement [ci skip] | Sam Auciello | 2015-08-24 | 1 | -2/+2 |
| | | |||||
* | | temp files are no more require | Gaurav Sharma | 2015-08-25 | 1 | -2/+0 |
| | | | | | | `:nail_care:` | ||||
* | | Merge branch 'master' of github.com:rails/rails | Vijay Dev | 2015-08-24 | 43 | -278/+884 |
|\ \ | | | | | | | | | | | | | Conflicts: guides/source/security.md | ||||
| * \ | Merge pull request #16245 from byroot/more-atomic-write | Matthew Draper | 2015-08-21 | 1 | -27/+29 |
| |\ \ | | | | | | | | | File renaming should be the last operation of an atomic write | ||||
| | * | | File renaming should be the last operation of an atomic write | Jean Boussier | 2015-08-20 | 1 | -27/+29 |
| | | | | |||||
| * | | | Merge pull request #21300 from jonahb/cache | Yves Senn | 2015-08-20 | 1 | -1/+1 |
| |\ \ \ | | | | | | | | | | | Correct cache store superclass in comment [ci skip] | ||||
| | * | | | Correct cache store superclass in comment | Jonah Burke | 2015-08-19 | 1 | -1/+1 |
| | | |/ | | |/| | |||||
| * | | | Merge pull request #21279 from ronakjangir47/test_cleanup | Kasper Timm Hansen | 2015-08-20 | 2 | -2/+10 |
| |\ \ \ | | |/ / | |/| | | Cleaned up generators tests using internal assertion helper | ||||
| | * | | Cleaned up generators tests using internal assertion helper | Ronak Jangir | 2015-08-20 | 2 | -2/+10 |
| | |/ | |||||
| * / | Use == 0 instead of .zero? in #try | Jean Boussier | 2015-08-17 | 1 | -1/+1 |
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The perf gain is relatively minor but consistent: ``` Calculating ------------------------------------- 0.zero? 137.091k i/100ms 1.zero? 137.350k i/100ms 0 == 0 142.207k i/100ms 1 == 0 144.724k i/100ms ------------------------------------------------- 0.zero? 8.893M (± 6.5%) i/s - 44.280M 1.zero? 8.751M (± 6.4%) i/s - 43.677M 0 == 0 10.033M (± 7.0%) i/s - 49.915M 1 == 0 9.814M (± 8.0%) i/s - 48.772M ``` And try! is quite a big hotspot for us so every little gain is appreciable. | ||||
| * | Replacing lambda with proc getting argument error because of it. | Ronak Jangir | 2015-08-17 | 2 | -1/+7 |
| | | |||||
| * | Merge pull request #20647 from twalpole/method_source_dependency | Rafael Mendonça França | 2015-08-14 | 1 | -0/+1 |
| |\ | | | | | | | Add method_source dependency to activesupport | ||||
| | * | Add method_source dependency to activesupport | Thomas Walpole | 2015-07-29 | 1 | -0/+1 |
| | | | |||||
| * | | Merge pull request #21025 from ronakjangir47/asserts | Kasper Timm Hansen | 2015-08-13 | 2 | -1/+18 |
| |\ \ | | | | | | | | | Added helper methods to stub any instance | ||||
| | * | | Added helper methods to stub any instance | Ronak Jangir | 2015-08-13 | 2 | -1/+18 |
| | | | | |||||
| * | | | Merge pull request #21219 from repinel/assert-human-size-prefix-deprecated | Yves Senn | 2015-08-13 | 1 | -8/+10 |
| |\ \ \ | | | | | | | | | | | Assert that the `:prefix` option of `number_to_human_size` is deprecated | ||||
| | * | | | Assert that the `:prefix` option of `number_to_human_size` is deprecated | Roque Pinel | 2015-08-12 | 1 | -8/+10 |
| | | | | | |||||
| * | | | | Merge pull request #21217 from myrridin/myrridin-documentation-updates | Zachary Scott | 2015-08-12 | 2 | -4/+4 |
| |\ \ \ \ | | |/ / / | |/| | | | [ci skip] Documentation: Switch around a common phrase for readability | ||||
| | * | | | [ci skip] Switch around a common idiom for readability | Thomas Hart II | 2015-08-05 | 2 | -4/+4 |
| | | |/ | | |/| | |||||
| * | | | [ci skip] Fix rdoc markup | akihiro17 | 2015-08-12 | 1 | -1/+1 |
| | |/ | |/| | |||||
| * | | :nodoc: internal class [ci skip] | Godfrey Chan | 2015-08-10 | 1 | -1/+1 |
| | | | |||||
| * | | Deprecate :si prefix in number_to_human_size without replacement | Jean Boussier | 2015-08-10 | 2 | -9/+15 |
| | | | |||||
| * | | Deprecate the :prefix option of `number_to_human_size` | Jean Boussier | 2015-08-10 | 2 | -2/+4 |
| | | | |||||
| * | | Merge pull request #21124 from kirs/feature/reload-i18n | Kasper Timm Hansen | 2015-08-10 | 1 | -2/+17 |
| |\ \ | | | | | | | | | Reload I18n.load_path in development | ||||
| | * | | Reload I18n locales in development | Kir Shatrov | 2015-08-10 | 1 | -2/+17 |
| | | | | |||||
| * | | | Merge pull request #21173 from repinel/fix-callback-terminator-docs | Claudio B. | 2015-08-08 | 1 | -3/+3 |
| |\ \ \ | | | | | | | | | | | [ci skip] Fix the AS::Callbacks terminator docs | ||||
| | * | | | [ci skip] Fix the AS::Callbacks terminator docs | Roque Pinel | 2015-08-08 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The second argument of the terminator lambda is no longer the result of the callback, but the result lambda. https://github.com/rails/rails/blob/3a7609e2bafee4b071fe35136274e6ccbae8cacd/activesupport/test/callbacks_test.rb#L553 | ||||
| * | | | | replace each with each_key when only the key is needed | Aaron Lasseigne | 2015-08-08 | 1 | -1/+1 |
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Using each_key is faster and more intention revealing. Calculating ------------------------------------- each 31.378k i/100ms each_key 33.790k i/100ms ------------------------------------------------- each 450.225k (± 7.0%) i/s - 2.259M each_key 494.459k (± 6.3%) i/s - 2.467M Comparison: each_key: 494459.4 i/s each: 450225.1 i/s - 1.10x slower | ||||
| * | | | Regexp#=== is faster than String#=~ | schneems | 2015-08-07 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | Discussion https://github.com/JuanitoFatas/fast-ruby/pull/59#issuecomment-128513763 | ||||
| * | | | Only invoke the default block for mattr_accessor once so that it does not ↵ | Lachlan Sylvester | 2015-08-07 | 2 | -1/+7 |
| | | | | | | | | | | | | | | | | cause issues if it is not idempotent |