aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | "Use assert_nil if expecting nil. This will fail in minitest 6."Akira Matsuda2016-12-2569-196/+196
| | | | | | | | | |
* | | | | | | | | | public methods can be placed after private ones if you want toAkira Matsuda2016-12-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | | | | Abuse of protected in guidesAkira Matsuda2016-12-253-5/+5
| | | | | | | | | |
* | | | | | | | | | Privatize unneededly protected methods in RailtiesAkira Matsuda2016-12-2525-135/+136
| | | | | | | | | |
* | | | | | | | | | Privatize some methods that were originally protectedAkira Matsuda2016-12-251-25/+27
| | | | | | | | | |
* | | | | | | | | | Prefer class << self; def over def self.Akira Matsuda2016-12-251-200/+200
|/ / / / / / / / /
* | | | | | | | | AR::ConnectionAdapters::PostgreSQL::Name#parts is called from other instancesAkira Matsuda2016-12-251-4/+6
| | | | | | | | |
* | | | | | | | | `protected` here doesn't protect anythingAkira Matsuda2016-12-251-26/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | there aren't any instance method defined in this class
* | | | | | | | | Shuffle travis matrixMatthew Draper2016-12-251-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Run actionpack and actioncable tests together * Demote ac:integration to allow_failures -- it works most of the time, but still fails too frequently to rely upon
* | | | | | | | | Privatize unneededly protected methods in Active RecordAkira Matsuda2016-12-2426-158/+145
| | | | | | | | |
* | | | | | | | | No need to nodoc private methodsAkira Matsuda2016-12-2411-31/+31
| | | | | | | | |
* | | | | | | | | Shave a couple of allocations off Journey scan & parseMatthew Draper2016-12-254-203/+208
| | | | | | | | |
* | | | | | | | | self. is not needed when calling its own instance methodAkira Matsuda2016-12-241-2/+1
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Action ViewAkira Matsuda2016-12-247-39/+32
| | | | | | | | |
* | | | | | | | | No need to nodoc private methodsAkira Matsuda2016-12-249-20/+20
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Action CableAkira Matsuda2016-12-245-25/+24
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected method in Active JobAkira Matsuda2016-12-241-2/+2
| | | | | | | | |
* | | | | | | | | No need to nodoc private methodsAkira Matsuda2016-12-241-5/+5
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Action MailerAkira Matsuda2016-12-244-27/+20
| | | | | | | | |
* | | | | | | | | No need to nodoc private methodAkira Matsuda2016-12-241-1/+1
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Action PackAkira Matsuda2016-12-2415-83/+82
| | | | | | | | |
* | | | | | | | | No need to nodoc private methodsAkira Matsuda2016-12-245-24/+24
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Active ModelAkira Matsuda2016-12-246-27/+23
| | | | | | | | |
* | | | | | | | | No need to :nodoc: private methodsAkira Matsuda2016-12-242-6/+6
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Active SupportAkira Matsuda2016-12-2415-56/+46
| | | | | | | | |
* | | | | | | | | No need to nodoc private methodsAkira Matsuda2016-12-245-10/+10
| | | | | | | | |
* | | | | | | | | Wrong rake task name in the documentAkira Matsuda2016-12-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip]
* | | | | | | | | bundle u rake to 12.0.0Akira Matsuda2016-12-241-1/+1
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Active Support testsAkira Matsuda2016-12-247-8/+8
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Railties testsAkira Matsuda2016-12-246-6/+6
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Active Record testsAkira Matsuda2016-12-2411-11/+11
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Action Cable testsAkira Matsuda2016-12-243-3/+3
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Active Job testsAkira Matsuda2016-12-242-2/+2
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Action Mailer testsAkira Matsuda2016-12-243-3/+3
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Action Pack testsAkira Matsuda2016-12-2310-22/+20
| | | | | | | | |
* | | | | | | | | Privatize unneededly protected methods in Action View testsAkira Matsuda2016-12-235-5/+5
| | | | | | | | |
* | | | | | | | | Describe what we are protectingAkira Matsuda2016-12-2337-0/+78
| | | | | | | | |
* | | | | | | | | Unneeded attr_accessor with side effectsAkira Matsuda2016-12-232-8/+0
|/ / / / / / / /
* | | | | | | | Merge pull request #27424 from utilum/fix_complex_and_rational_are_duplicableKasper Timm Hansen2016-12-223-2/+27
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix duplicable? for Complex and Rational
| * | | | | | | | Fix Complex and Rational are duplicable?utilum2016-12-213-2/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | See [this test](https://gist.github.com/utilum/78918f1b64f8b61ee732cb266db7c43a).
* | | | | | | | | Merge pull request #27431 from y-yagi/quiet_generator_log_in_testKasper Timm Hansen2016-12-221-1/+1
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | quiet generators log in test
| * | | | | | | | | quiet generators log in testyuuji.yaginuma2016-12-221-1/+1
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This quiet the following log. ``` create app/mailers/notifier_mailer.rb invoke erb create app/views/notifier_mailer identical app/views/layouts/mailer.text.erb identical app/views/layouts/mailer.html.erb create app/views/notifier_mailer/foo.text.erb create app/views/notifier_mailer/foo.html.erb invoke test_unit create test/mailers/notifier_mailer_test.rb create test/mailers/previews/notifier_mailer_preview.rb ```
* | | | | | | | | Merge pull request #27430 from kirs/aj-warningMatthew Draper2016-12-221-1/+1
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | Remove warning in ActiveJob
| * | | | | | | | Remove warning in ActiveJobKir Shatrov2016-12-211-1/+1
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently it causes: ``` activejob/Rakefile:5: warning: already initialized constant ACTIVEJOB_ADAPTERS ```
* | | | | | | | Merge pull request #27427 from rails/binary-paramsAaron Patterson2016-12-216-54/+56
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Document and update API for `skip_parameter_encoding`
| * | | | | | | | updating docsAaron Patterson2016-12-211-0/+4
| | | | | | | | |
| * | | | | | | | Document and update API for `skip_parameter_encoding`Aaron Patterson2016-12-216-54/+52
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit changes `parameter_encoding` to `skip_parameter_encoding`. `skip_parameter_encoding` will set encoding on all parameters to ASCII-8BIT for a given action on a particular controller. This allows the controller to handle data when the encoding of that data is unknown, for example file systems or truly binary parameters.
* / / / / / / / fix typo in getting_started [ci skip] (#27423)yachibit2016-12-211-1/+1
|/ / / / / / /
* | | | | | | Merge pull request #27355 from yukideluxe/fixtures-deleted-tablesKasper Timm Hansen2016-12-216-3/+28
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | fix fixtures loading for models with same table names in different databases
| * | | | | | | fix #create_fixtures when equal table names in different databasesJulia Lopez2016-12-216-3/+28
|/ / / / / / /