47 commits

Author	SHA1	Message	Date
Sijawusz Pur Rahnama	82e0e53080	Revert "Fix Performance/ExcessiveAllocations to exclude each calls without a block" ... This reverts commit 29e29b8e1d.	2023-11-17 19:48:41 +01:00
Sijawusz Pur Rahnama	10b577d23a	Use square brackets for %w and %i array literals	2023-11-17 17:34:39 +01:00
Sijawusz Pur Rahnama	f3f1f3a2ab	Misc cleanups/refactors	2023-11-14 12:34:44 +01:00
Sijawusz Pur Rahnama	f984d83b05	Use name(_end)_location helpers consistently	2023-11-12 10:24:12 +01:00
Sijawusz Pur Rahnama	98cc6fd612	Make it easier to add issues for nodes with name location preference	2023-11-12 10:23:36 +01:00
Sijawusz Pur Rahnama	d5ac394d19	Switch only FIXME comment to TODO	2023-11-06 16:59:09 +01:00
Sijawusz Pur Rahnama	ddff8d226b	Add Performance/MinMaxAfterMap rule	2023-07-10 15:46:17 +02:00
Sijawusz Pur Rahnama	5cff76071a	No need for such micro-optimizations, LLVM takes care of those	2023-07-02 14:34:25 +02:00
Sijawusz Pur Rahnama	29e29b8e1d	Fix Performance/ExcessiveAllocations to exclude each calls without a block	2023-06-30 21:44:47 +02:00
Sijawusz Pur Rahnama	abe5237802	Add Performance/ExcessiveAllocations rule	2023-06-30 15:17:40 +02:00
Sijawusz Pur Rahnama	16141a376e	Cleanup properties definition macro	2023-06-14 15:08:19 +02:00
Sijawusz Pur Rahnama	38b6751bc0	Add AST::NodeVisitor::Category simplifying code a bit	2023-06-12 23:17:14 +02:00
Sijawusz Pur Rahnama	6f0b6ffcd0	Remove buggy auto-correction from Performance/AnyInsteadOfEmpty rule	2023-01-04 02:40:07 +01:00
Sijawusz Pur Rahnama	07ce595ef2	Readability refactors	2022-12-08 17:47:22 +01:00
Sijawusz Pur Rahnama	d942524efa	Remove trailing dots from rule descriptions	2022-10-30 21:53:09 +01:00
Sijawusz Pur Rahnama	fbd72a5171	Fix regression in AnyInsteadOfEmpty rule	2022-04-22 00:58:37 +02:00
fn ⌃ ⌥	63a6c73dc0	Autocorrect various rules (#253)	2021-11-16 22:30:33 +01:00
Vitalii Elenhaupt	9e2d4f1856	Disable performance rules for spec files ... closes #220	2021-04-18 09:39:55 +03:00
Sijawusz Pur Rahnama	788e0a3fad	Enable MapInsteadOfBlock rule again	2021-04-12 06:23:03 +02:00
Sijawusz Pur Rahnama	2694a0c865	Remove “join” from the list of trigger methods	2021-04-12 06:22:54 +02:00
Vitalii Elenhaupt	0881b861d7	Disable MapInsteadOfBlock by default ... refs #208	2021-03-29 09:15:18 +03:00
Vitalii Elenhaupt	3a1ee7fa21	Correct yml examples	2021-02-11 11:01:30 +02:00
Sijawusz Pur Rahnama	7b3c814914	Fix newly found issues	2021-01-28 01:03:48 +01:00
Sijawusz Pur Rahnama	d71091a40c	Add Performance/AnyInsteadOfEmpty rule	2021-01-28 01:03:48 +01:00
Sijawusz Pur Rahnama	6acb8ad2eb	ChainedCallsWithNoBang -> ChainedCallWithNoBang	2021-01-22 17:24:45 +01:00
Sijawusz Pur Rahnama	8babeea80f	Add repeated_combinations and repeated_permutations	2021-01-22 12:21:14 +01:00
Sijawusz Pur Rahnama	a219a73258	Remove rotate from the list of call_names ... In some cases it returns `self` and not a copy.	2021-01-22 12:20:50 +01:00
Sijawusz Pur Rahnama	d3b952f58a	Add Performance/ChainedCallsWithNoBang rule	2021-01-21 12:47:10 +01:00
Sijawusz Pur Rahnama	ad8f570a04	Change Rule to class	2021-01-19 22:17:17 +01:00
Sijawusz Pur Rahnama	ddaa58f475	Merge pull request #192 from crystal-ameba/feature/flatten-after-map-rule ... Add Performance/FlattenAfterMap rule	2021-01-19 22:16:10 +01:00
Sijawusz Pur Rahnama	23b4b4c4f0	Add Performance/FlattenAfterMap rule	2021-01-18 18:04:54 +01:00
Sijawusz Pur Rahnama	b7286dc673	Add Performance/CompactAfterMap rule	2021-01-18 18:04:12 +01:00
Sijawusz Pur Rahnama	d2fa75280f	Extend JoinAfterMap to check also calls to sum/product and rename it to MapInsteadOfBlock (#190)	2021-01-18 17:42:50 +02:00
Sijawusz Pur Rahnama	af5d825015	Followup to #185	2021-01-17 18:45:14 +01:00
Sijawusz Pur Rahnama	5e58a60ade	Move constant definitions after properties call	2021-01-17 17:39:32 +01:00
Sijawusz Pur Rahnama	a892cd43b0	Add Performance/JoinAfterMap rule	2021-01-17 16:57:15 +01:00
Sijawusz Pur Rahnama	06852c490b	Followup to #185	2021-01-17 13:25:50 +01:00
Sijawusz Pur Rahnama	1a091c1f1a	Optimize rules (#185) ... * Replace pointless interpolations with actual values * Rules optimizations * Stylistic refactors * Remove extraneous blank lines * Remove some instances of .not_nil! usage	2021-01-17 14:16:04 +02:00
Vitalii Elenhaupt	1a0468c653	Use NodeVisitor by default and remove the boilerplace code	2020-03-25 18:21:07 +02:00
Vitalii Elenhaupt	037d1dde74	Performance/FirstLastAfterFilter: ignore .first/.last with args ... closes #104	2019-06-03 18:25:48 +03:00
Vitalii Elenhaupt	94e1d4567a	Properly report performance rules in macros ... closes #102	2019-05-05 18:05:08 +03:00
V. Elenhaupt	fd78d17c40	New rule: Performance/FirstLastAfterFilter (#76)	2018-09-06 17:59:11 +03:00
Vitalii Elenhaupt	a1b34eb7be	New rule: Performance/AnyAfterFilter	2018-09-03 22:55:10 +03:00
Vitalii Elenhaupt	790b519653	Performance/Count -> Performance/SizeAfterFilter	2018-09-03 22:44:30 +03:00
Vitalii Elenhaupt	799c0fd5e1	Correct location name of reported issue	2018-09-03 13:26:19 +03:00
Vitalii Elenhaupt	9fa13848bf	Correct doc example for Count rule	2018-09-03 12:00:23 +03:00
Vitalii Elenhaupt	446f557c23	New rule: Performance/Count	2018-09-03 01:07:34 +03:00