aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | Simplify and fix grammarJon Atack2014-11-141-5/+4
|/ / / | | | | | | [skip ci]
* | | Merge pull request #17620 from arunagw/aa-build-fix-isolatedSean Griffin2014-11-1430-0/+31
|\ \ \ | | | | | | | | Build fix when running in isolation
| * | | Build fix when running in isolationArun Agrawal2014-11-1430-0/+31
| | | | | | | | | | | | | | | | | | | | `Computer` class needs to be require See #17217 for more details
* | | | Use `DelegateClass` instead of `SimpleDelegator` for type decoratorsSean Griffin2014-11-142-2/+2
| | | | | | | | | | | | | | | | There is a significant performance difference between the two. Closes
* | | | Merge pull request #17614 from gsamokovarov/exception-wrapper-basic-testsSantiago Pastorino2014-11-141-0/+66
|\ \ \ \ | | | | | | | | | | Add basic tests for ActionDispatch::ExceptionWrapper
| * | | | Add basic tests for ActionDispatch::ExceptionWrapperGenadi Samokovarov2014-11-131-0/+66
| | | | |
* | | | | Merge pull request #17619 from tgxworld/fixes_to_testing_guideZachary Scott2014-11-141-2/+2
|\ \ \ \ \ | |_|/ / / |/| | | | [ci skip] Corrections to testing guide.
| * | | | [CI SKIP] Corrections to testing guide.Guo Xiang Tan2014-11-141-2/+2
|/ / / / | | | | | | | | | | | | Partially revert https://github.com/rails/rails/commit/c31e638a37946a120f6dbf207127d36beea5ab85.
* | | | Merge pull request #17618 from sandipransing/testing-guideArun Agrawal2014-11-141-4/+2
|\ \ \ \ | | | | | | | | | | [ci skip] Remove deprecated testing tasks
| * | | | [ci skip] Remove deprecated testing tasksSandip Ransing2014-11-141-4/+2
|/ / / / | | | | | | | | | | | | | | | | This has changed recently and so rake test:all and rake test:all:db are deprecated.
* | | | exec_prepared is GVL friendly, so lets use it.Aaron Patterson2014-11-132-5/+3
| | | | | | | | | | | | | | | | | | | | also increase the version of pg required so that people will get the GVL friendly version
* | | | Merge pull request #17613 from sandipransing/testing-guideAbdelkader Boudih2014-11-131-3/+5
|\ \ \ \ | | | | | | | | | | [ci skip] Improve upon testing tasks guide
| * | | | [ci skip] Improve upon testing tasks guideSandip Ransing2014-11-141-3/+5
|/ / / / | | | | | | | | | | | | Add missing testing tasks and also improve upon tasks descriptions
* | | | Merge pull request #17612 from sandipransing/testing-guideRafael Mendonça França2014-11-131-0/+1
|\ \ \ \ | | | | | | | | | | [ci skip] Capybara added to other testing approaches
| * | | | [ci skip] Capybara added to other testing approachesSandip Ransing2014-11-141-0/+1
|/ / / /
* | | | Merge branch 'sandipransing-master'Zachary Scott2014-11-131-2/+6
|\ \ \ \
| * | | | [ci skip] Add one more example to routing testing, and highlight associationSandip Ransing2014-11-131-2/+6
|/ / / / | | | | | | | | | | | | name keywords
* | | | Merge pull request #17604 from rymohr/message-verifier-case-sensitivitySantiago Pastorino2014-11-131-2/+10
|\ \ \ \ | | | | | | | | | | Abstract encoding strategy for ActiveSupport::MessageVerifier
| * | | | Abstract encoding strategy for ActiveSupport::MessageVerifierRyan Mohr2014-11-121-2/+10
|/ / / /
* | | | Improve railties changelog entry and move latest to the top [ci skip]Carlos Antonio da Silva2014-11-121-8/+7
| | | |
* | | | Merge pull request #17593 from igas/paths-unshift-arityRafael Mendonça França2014-11-122-2/+7
|\ \ \ \ | | | | | | | | | | Fix Rails::Paths::Path.unshift interface
| * | | | Fix Rails::Paths::Path.unshift interfaceIgor Kapkov2014-11-122-2/+7
| | | | |
* | | | | Merge pull request #17596 from aderyabin/follow_convensionRafael Mendonça França2014-11-121-1/+1
|\ \ \ \ \ | |_|_|/ / |/| | | | Follow the coding conventions
| * | | | Follow the coding conventionsAndrey Deryabin2014-11-121-1/+1
|/ / / /
* | | | Merge pull request #17348 from DavidGeukers/rake_test_allDavid Heinemeier Hansson2014-11-123-6/+28
|\ \ \ \ | |/ / / |/| | | simplify rake test vs rake test:all
| * | | Simplify rake test vs rake test:allDavid Geukers2014-11-103-6/+28
| | | | | | | | | | | | | | | | Renames `rake test:all` to `rake test` by changing old `rake test:run` to previous version of `rake test:all`. Removes old definition of `rake test`. Also renames `rake test:all:db` to `rake test:db` and deprecates `rake test:all` & `rake test:all:db`
* | | | Merge pull request #17588 from djpowers/patch-1Zachary Scott2014-11-111-2/+2
|\ \ \ \ | | | | | | | | | | [ci skip] Use American English and replace "behaviour" with "behavior".
| * | | | Replace "behaviour" with "behavior"Dave Powers2014-11-111-2/+2
|/ / / / | | | | | | | | Update to consistently use American English [ci skip]
* | | | Merge pull request #17047 from betesh/masterRafael Mendonça França2014-11-111-0/+1
|\ \ \ \ | | | | | | | | | | Time includes DateAndTime::Zones acts_like(:time)
| * | | | Time includes DateAndTime::Zones acts_like(:time)betesh2014-11-091-0/+1
| | | | |
* | | | | Merge pull request #17580 from ccutrer/change_table_nameYves Senn2014-11-113-17/+29
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | add a Table#name accessor like TableDefinition#name
| * | | | | add a Table#name accessor like TableDefinition#nameCody Cutrer2014-11-103-17/+29
| | | | | |
* | | | | | Make FlashHash#key? work with symbol and stringRafael Mendonça França2014-11-112-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | Closes #17586
* | | | | | Remove unnecessary double spaceclaudiob2014-11-102-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replaces the following in two places: ```diff -require ::File.expand_path('../config/environment', __FILE__) +require ::File.expand_path('../config/environment', __FILE__) ```
* | | | | | Merge branch 'rm-fix-secret-token-fallback'Rafael Mendonça França2014-11-109-23/+143
|\ \ \ \ \ \
| * | | | | | :scissors:Rafael Mendonça França2014-11-101-1/+1
| | | | | | |
| * | | | | | No need to sync config.secret_token and secrets.secret_tokenRafael Mendonça França2014-11-102-41/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Just prefer secrets over config
| * | | | | | Merge pull request #16535 from bf4/patch-1Rafael Mendonça França2014-11-109-23/+180
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | Allow fallback to LegacyKeyGenerator when secret_key_base is not set but secrets.secret_token is
| | * | | | | `secret_token` is now saved in `Rails.application.secrets.secret_token`Benjamin Fleischer2014-11-029-23/+180
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - `secrets.secret_token` is now used in all places `config.secret_token` was - `secrets.secret_token`, when not present in `config/secrets.yml`, now falls back to the value of `config.secret_token` - when `secrets.secret_token` is set, it over-writes `config.secret_token` so they are the same (for backwards-compatibility) - Update docs to reference app.secrets in all places - Remove references to `config.secret_token`, `config.secret_key_base` - Warn that missing secret_key_base is deprecated - Add tests for secret_token, key_generator, and message_verifier - the legacy key generator is used with the message verifier when secrets.secret_key_base is blank and secret_token is set - app.key_generator raises when neither secrets.secret_key_base nor secret_token are set - app.env_config raises when neither secrets.secret_key_base nor secret_token are set - Add changelog Run focused tests via ruby -w -Itest test/application/configuration_test.rb -n '/secret_|key_/'
* | | | | | | dependencies.rb: keep the decorated #load and #require private [closes #17553]Xavier Noria2014-11-103-13/+36
| | | | | | |
* | | | | | | Merge pull request #17585 from aderyabin/remove_limited_update_conditionsRafael Mendonça França2014-11-102-8/+0
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | remove never called method `limited_update_conditions`
| * | | | | | remove never called method `limited_update_conditions`Andrey Deryabin2014-11-112-8/+0
| | | | | | |
* | | | | | | Merge pull request #17579 from aderyabin/cleanup_sqliteRafael Mendonça França2014-11-101-6/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Tiny improvement in sqlite3 adapter
| * | | | | | | tiny code improvement in sqlite3 adapter:Andrey Deryabin2014-11-101-6/+2
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | - remove unused method `supports_add_column?` - change additional restriction method to `valid_alter_table_type?` - fix code style
* | | | | | | Merge pull request #17578 from codeodor/fix-17217-test-bugRafael Mendonça França2014-11-101-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Fix bug found when running individual tests against #17217 after merging
| * | | | | | | Fixes a bug found when running individual tests against #17217 after mergingSammy Larbi2014-11-101-1/+1
| |/ / / / / /
* | | | | | | Merge pull request #11694 from ↵Rafael Mendonça França2014-11-103-1/+20
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Empact/association-bind-values-not-updated-on-save Fix that a collection proxy could be cached before the save of the owner, resulting in an invalid proxy lacking the owner’s id Conflicts: activerecord/CHANGELOG.md
| * | | | | | Fix that a collection proxy could be cached before the save of the owner, ↵Ben Woosley2014-10-033-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | resulting in an invalid proxy lacking the owner’s id. Absent this fix calls like: owner.association.update_all to behave unexpectedly because they try to act on association objects where owner_id is null. more evidence here: https://gist.github.com/Empact/5865555 ``` Active Record 3.2.13 -- create_table(:firms, {:force=>true}) -> 0.1371s -- create_table(:clients, {:force=>true}) -> 0.0005s 1 clients. 1 expected. 1 clients updated. 1 expected. ``` ``` Active Record 4.0.0 -- create_table(:firms, {:force=>true}) -> 0.1606s -- create_table(:clients, {:force=>true}) -> 0.0004s 1 clients. 1 expected. 0 clients updated. 1 expected. ```
* | | | | | | docs, minor changes in Active Job documentation. [ci skip]Yves Senn2014-11-102-6/+5
| | | | | | |
* | | | | | | Remove useless `only_path: true` in path helpersGodfrey Chan2014-11-101-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We added a deprecation warning for these cases in aa1fadd, so these are now causing deprecation warnings in the test output. AFAICT, in these two cases, the option is not integral to the purpose of the test, so they can be safely removed
generated by cgit v1.2.3 (git 2.25.1) at 2025-03-22 23:49:19 +0000