aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #17232 from Agis-/issue-17209Rafael Mendonça França2014-10-134-1/+28
|\ | | | | Autosave callbacks shouldn't be `after_save` callbacks
| * Autosave callbacks shouldn't be `after_save`Agis-2014-10-134-1/+28
| | | | | | | | | | | | | | | | | | | | | | | | 068f092ced8483e557725542dd919ab7c516e567 registered autosave callbacks as `after_save` callbacks. This caused the regression described in #17209. Autosave callbacks should be registered as `after_update` and `after_create` callbacks, just like before. This is a partial revert of 068f092ced8483e557725542dd919ab7c516e567. Fixes #17209.
* | line_statistics is not an executableRafael Mendonça França2014-10-132-2/+2
| | | | | | | | Also add documentation on it
* | Make a note about the internal tools/ dir and what each utility doesJeremy Kemper2014-10-131-0/+8
| |
* | Do not use deprecated `exists?` methodRafael Mendonça França2014-10-131-1/+1
| |
* | Document the propose of tools/profileRafael Mendonça França2014-10-131-0/+3
| |
* | Revert "Merge pull request #17247 from igas/fix-deprecations"Rafael Mendonça França2014-10-131-0/+133
| | | | | | | | | | | | | | This reverts commit 997f1575f78bf6a2d12b8665a55807d51fe964df, reversing changes made to f919d5f5ea75f0e56a57643e5a0472367a98625f. See https://github.com/rails/rails/pull/17247#issuecomment-58895078
* | Merge pull request #17230 from robertoz-01/masterRafael Mendonça França2014-10-131-1/+1
|\ \ | | | | | | atomic_write rescue also Errno::EACCES
| * | atomic_write rescue also Errno::EACCESRoberto Zanon2014-10-101-1/+1
| | | | | | | | | atomic_write rescue also Errno::EACCES on changing file permission. It could be raised with some type of filesystem
* | | Merge pull request #17247 from igas/fix-deprecationsRafael Mendonça França2014-10-131-133/+0
|\ \ \ | | | | | | | | | | | | | | | | remove unused tools/profile We usually use moderns tools like benchmark-ips so we don't need this anymore
| * | | remove unused tools/profileIgor Kapkov2014-10-131-133/+0
| | | |
* | | | Merge pull request #17240 from masarakki/fix-test_helper-of-mountable-pluginRafael Mendonça França2014-10-132-0/+7
|\ \ \ \ | |/ / / |/| | | fix test_helper for mountable plugin
| * | | fix test_helper for mountable pluginmasarakki2014-10-132-0/+7
| | |/ | |/|
* | | Merge pull request #17234 from arthurnn/add_regression_for_path_and_cascadeRafael Mendonça França2014-10-131-0/+8
|\ \ \ | | | | | | | | Add regression test for router was overwriting PATH_INFO
| * | | Add regression test for router was overwriting PATH_INFOArthur Neves2014-10-101-0/+8
| |/ / | | | | | | | | | [related #17233]
* | | Merge pull request #17246 from sferik/railties_firstRafael Mendonça França2014-10-131-7/+7
|\ \ \ | | | | | | | | Start running railties builds first, since they take the longest finish
| * | | Start running railties builds first, since they take the longest finishErik Michaels-Ober2014-10-131-7/+7
| | | |
* | | | Merge pull request #17245 from sferik/sampleXavier Noria2014-10-131-2/+2
|\ \ \ \ | | | | | | | | | | Replace Array#shuffle.first with Array#sample
| * | | | Replace Array#shuffle.first with Array#sampleErik Michaels-Ober2014-10-131-2/+2
| |/ / /
* | | | Merge pull request #17244 from sferik/reverse_eachXavier Noria2014-10-135-6/+6
|\ \ \ \ | |/ / / |/| | | Replace Enumerable#reverse.each with Enumerable#reverse_each
| * | | Replace Enumerable#reverse.each with Enumerable#reverse_eachErik Michaels-Ober2014-10-135-6/+6
|/ / /
* | | Merge pull request #17241 from josh/coffee-rails-4-1Guillermo Iguaran2014-10-122-3/+3
|\ \ \ | |/ / |/| | Upgrade to coffee-rails 4.1.0
| * | Default generated Gemfile to coffee-rails ~> 4.1.0Joshua Peek2014-10-122-3/+3
|/ /
* | Merge pull request #17014 from grosser/grosser/fast-fixturesAaron Patterson2014-10-103-66/+2
|\ \ | |/ |/| speed up fixtures by not loading all their classes
| * speed up fixtures by not loading all their classesgrosser2014-10-063-66/+2
| |
* | tests, move `original_` var assignments outside of `begin`.Yves Senn2014-10-102-11/+16
| |
* | Merge pull request #17223 from bgentry/active-job-empty-queue-nameYves Senn2014-10-102-1/+21
|\ \ | | | | | | ActiveJob: allow empty queue names
| * | ActiveJob: allow blank queue namesBlake Gentry2014-10-092-1/+21
|/ / | | | | | | | | | | | | As discussed in #17195, Que uses blank queue names by default for performance reasons. At the very least, ActiveJob should allow users to choose "" as their queue name. This commit allows blank queue names, as well as tests to make sure that nil queue names still use the default.
* | Merge pull request #17221 from StefanH/fix-unknown-http-method-messageRafael Mendonça França2014-10-092-1/+17
|\ \ | | | | | | Invalid HTTP method will raise I18n::InvalidLocale if :en is not a valid locale.
| * | Request#check_method no longer breaks when :en is not available localeStefan Henzen2014-10-092-1/+17
|/ / | | | | | | | | | | | | | | Request#check_method would use to_sentence(locale: :en), which breaks when I18n.available_locales does not include :en and I18n.enforce_available_locales is true (default). Inlined to_sentence functionality to solve this.
* | Merge pull request #17219 from Gaurav2728/gaurav-missing_i18_linkAbdelkader Boudih2014-10-091-9/+6
|\ \ | | | | | | [ci skip] removing ruby-i18n.org/wiki link that is not available and usi...
| * | [ci skip] removing ruby-i18n.org/wiki link that is not available and using ↵Gaurav Sharma2014-10-091-9/+6
|/ / | | | | | | “gems” instead of plugins
* | Merge pull request #17215 from aripollak/sidekiq-no-retry-optionRafael Mendonça França2014-10-081-3/+1
|\ \ | | | | | | Don't pass retry option to Sidekiq
| * | Don't pass retry option to SidekiqAri Pollak2014-10-081-3/+1
|/ / | | | | | | | | * This would override whatever default is configured in Sidekiq system-wide, and retry is true by default anyway.
* | Point to a specific version of guide to make the redirect workRafael Mendonça França2014-10-081-1/+1
| |
* | Merge pull request #17205 from y-yagi/patch-1Abdelkader Boudih2014-10-081-1/+1
|\ \ | | | | | | update error message when InlineAdapter is used
| * | update error message when InlineAdapter is usedyuuji.yaginuma2014-10-081-1/+1
| | |
* | | Merge pull request #17187 from Bounga/middleware_reloading_explainationAbdelkader Boudih2014-10-081-0/+4
|\ \ \ | |/ / |/| | Warning about middleware not auto-reloading
| * | Warning about middleware not auto-reloadingNicolas Cavigneaux2014-10-081-0/+4
|/ / | | | | | | | | | | | | | | | | Add a section in the guide to explain that Rails can't auto-reload a middleware on code change. Fix #16806 [ci skip]
* | Only include the needed assertionsRafael Mendonça França2014-10-072-2/+3
| |
* | Merge pull request #17204 from schneems/schneems/fix-am-tests-masterRafael Mendonça França2014-10-074-7/+9
|\ \ | | | | | | | | | fix tests on action_mailer
| * | fix tests on action_mailerschneems2014-10-074-7/+9
| | | | | | | | | Include appropriate helpers and use `assert_dom_equal` where applicable
* | | Merge pull request #17203 from schneems/schneems/av-tag_helperRafael Mendonça França2014-10-073-10/+9
|\| | | | | | | | remove un-needed sort
| * | remove un-needed sortschneems2014-10-073-10/+9
|/ / | | | | | | | | | | HTML doesn't care what order the elements are rendered in, so why should we? Updates tests to use proper `assert_dom_equal` instead of `assert_equal` /cc @jeremy
* | Merge pull request #17201 from djpowers/patch-1Abdelkader Boudih2014-10-071-1/+1
|\ \ | | | | | | Fix grammar [ci skip]
| * | Fix grammarDave Powers2014-10-071-1/+1
|/ /
* | Merge pull request #17198 from neerajdotname/wordsmitthingRafael Mendonça França2014-10-071-2/+2
|\ \ | | | | | | Rephrasing sentences
| * | Rephrasing sentencesNeeraj Singh2014-10-071-2/+2
| | |
* | | Merge pull request #17184 from fillman/masterRafael Mendonça França2014-10-071-6/+17
|\ \ \ | | | | | | | | fix autoload tests
| * | | fix autoload testsfillman2014-10-051-6/+17
| | | |