rails.git - Mirror of official rails repo with custom fixes.

	Commit message (Collapse)	Author	Age	Files	Lines
*	Enable `Layout/EmptyLinesAroundAccessModifier` cop	Ryuta Kamizono	2019-06-13	1	-1/+0
\| \| \| \| \| \| \| \| \| \| \|	We sometimes say "✂️ newline after `private`" in a code review (e.g. https://github.com/rails/rails/pull/18546#discussion_r23188776, https://github.com/rails/rails/pull/34832#discussion_r244847195). Now `Layout/EmptyLinesAroundAccessModifier` cop have new enforced style `EnforcedStyle: only_before` (https://github.com/rubocop-hq/rubocop/pull/7059). That cop and enforced style will reduce the our code review cost.
*	Added 'detach_from' to 'ActiveSupport::Subscriber' to detach a subscriber ↵	sushant	2019-04-04	1	-0/+23
\| \| \| \|	from a namespace.
*	[Active Support] `rubocop -a --only Layout/EmptyLineAfterMagicComment`	Koichi ITO	2017-07-11	1	-0/+1
\|
*	Use frozen-string-literal in ActiveSupport	Kir Shatrov	2017-07-09	1	-0/+1
\|
*	Revert "Merge pull request #29540 from kirs/rubocop-frozen-string"	Matthew Draper	2017-07-02	1	-1/+0
\| \| \| \| \|	This reverts commit 3420a14590c0e6915d8b6c242887f74adb4120f9, reversing changes made to afb66a5a598ce4ac74ad84b125a5abf046dcf5aa.
*	Enforce frozen string in Rubocop	Kir Shatrov	2017-07-01	1	-0/+1
\|
*	normalizes indentation and whitespace across the project	Xavier Noria	2016-08-06	1	-3/+3
\|
*	applies new string literal convention in activesupport/test	Xavier Noria	2016-08-06	1	-2/+2
\| \| \| \| \|	The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default.
*	Fix assertion arguments order	Carlos Antonio da Silva	2014-07-31	1	-1/+1
\|
*	Remove warning remeving the method before redefining	Rafael Mendonça França	2014-05-23	1	-0/+1
\| \| \| \| \| \| \|	We need to test if the same method defined more than once only register one subscriber for it. We can safelly remove because the method body is the same and Subscriber use method_added hook for register the subscriber.
*	Fixed duplicate subscribers in ActiveSupport::Subscriber	Dennis Schoen	2014-05-09	1	-6/+19
\| \| \| \| \|	ActiveSupport::Subscriber no longer creates multiple subscribers when you redefine a method.
*	Allow attaching to AS::Notifications namespace up front	Daniel Schierbeck	2013-09-20	1	-0/+40
	Before, you were required to attach after adding the methods to the class, since the attachment process needed the methods to be present. With this change, any new method will also be attached to the configured namespace.