Commit Graph

230 Commits

Author SHA1 Message Date
Marcelo Fabri d9ae32bfd1 Use Set for aliases in RuleDescription init 2017-01-02 16:23:03 -02:00
Marcelo Fabri c0aacd6adb Log deprecation warns for commands 2017-01-02 16:23:03 -02:00
Marcelo Fabri a5683fa393 PR feedback 2017-01-02 16:21:10 -02:00
Marcelo Fabri 2e8ada85b1 Logging deprecation warning and adding tests 2017-01-02 16:21:10 -02:00
Marcelo Fabri 07b14d852f Handle duplicated alias configuration 2017-01-02 16:21:10 -02:00
Marcelo Fabri 2e9e016723 Running "disable" tests on all aliases 2017-01-02 16:21:10 -02:00
Marcelo Fabri df8429d2eb Add support for rule aliases 2017-01-02 16:21:10 -02:00
JP Simard 253e1f4f86 make recording benchmarks thread-safe 2016-12-31 17:24:43 -08:00
JP Simard d82ca686d2 parallelize getting violations for a single file 2016-12-30 23:08:26 -08:00
Marcelo Fabri 7056434992 Add object_literal rule
Fixes #1060
2016-12-30 17:57:30 -02:00
Marcelo Fabri aea1fd3538 Fix false positives on closure rules with Swift 2.3
Fixes #1019
2016-12-29 18:18:06 -02:00
Marcelo Fabri 570e627f9a Sorted imports (#1071)
Add rule requiring imports be sorted
2016-12-26 18:28:43 -02:00
Marcelo Fabri c43f808cfc Re-enable ConfigurationTests.testExcludedPaths 2016-12-26 13:32:13 -02:00
JP Simard 27d1a49241 Merge pull request #1070 from marcelofabri/redundant_void_return
Add redundant_void_return rule
2016-12-25 22:57:25 -08:00
JP Simard 8fd337727b improve wording when SourceKit fails 2016-12-25 22:52:19 -08:00
Marcelo Fabri c6ac9b919c Add redundant_void_return rule
Fixes #1066
2016-12-26 01:01:04 -02:00
Marcelo Fabri 5c39a76526 Add redundant_optional_initialization rule
Fixes #1052
2016-12-24 17:28:45 -02:00
Marcelo Fabri 608a2725c8 Add class_delegate_protocol rule
Fixes #1039
2016-12-23 02:37:44 -02:00
Marcelo Fabri 64a624ccf6 Add vertical_parameter_alignment rule
Fixes #1033
2016-12-22 21:21:23 -02:00
Marcelo Fabri b18c98084c Fix existing violations 2016-12-22 11:57:06 -02:00
Marcelo Fabri 56f6fcdd5d Add first_where rule
Fixes #1005
2016-12-22 11:57:06 -02:00
Marcelo Fabri 367bbb84ff Merge pull request #1010 from marcelofabri/120-is-better-than-100
Changing the default line length limit to 120
2016-12-22 00:00:27 -02:00
JP Simard 8858b3c484 sort imports in Location.swift 2016-12-21 13:26:55 -08:00
Jamie Edge 9c8939a9db Resolve build issue by including missing import. Resolve line length violation. 2016-12-21 12:02:59 +00:00
Jamie Edge b9afeba7fd Switch to a relative path within the HTML report output. 2016-12-21 11:56:33 +00:00
Marcelo Fabri 5cccc12eb7 Style changes because of the 120 length limit 2016-12-20 12:10:11 -02:00
Marcelo Fabri 8102c59a53 Initial implementation of closure_end_indentation 2016-12-19 20:08:17 -02:00
Marcelo Fabri 6577b59cd7 Add unused_enumerated rule 2016-12-18 00:22:33 -02:00
Marcelo Fabri 3d7ce6ae8c Add UnusedClosureParameterRule 2016-12-15 19:32:44 -02:00
Marcelo Fabri 73c7d9a054 Add operator_usage_whitespace
Fixes #626
2016-12-15 10:37:03 -02:00
Marcelo Fabri d465d7840e Add empty_parameters rule
Fixes #573
2016-12-13 18:05:38 -02:00
Marcelo Fabri e786f676eb Add void_return rule
Fixes #964
2016-12-13 10:41:08 -02:00
JP Simard 11bfa938ae Merge pull request #960 from marcelofabri/closure_parameter_position
Add closure_parameter_position rule
2016-12-13 01:33:00 -08:00
Aaron McTavish f54c9c4d5a Add ProhibitedSuperRule
Resolves #970: `Rule: Methods calling super that should not`
2016-12-13 07:22:11 +00:00
Marcelo Fabri a48097103a Add closure_parameter_position rule
Fixes #931
2016-12-12 21:12:45 -02:00
Marcelo Fabri 9c43a21fed Merge branch 'master' into attributes-rule 2016-12-11 23:46:18 -02:00
JP Simard 5b98fbe2f6 make Command.swift compile on Linux
by addressing differences in APIs between Scanner on Linux and Darwin.
2016-12-11 14:04:49 -08:00
JP Simard 0faf567931 use bridge() rather than explict or implicit casts
this is a step in aligning the code to compile on both Darwin and
Linux
2016-12-11 13:47:44 -08:00
Daniel Duan b9835c3828 Add rule to ban dynamic inline functions
When a class method is marked as both `dynamic` and `@inline(__always)`, its
dispatch behavior is not well defined. Therefore this specific combination
should be discouraged.
2016-12-11 01:52:39 -08:00
Marcelo Fabri 0209c461b3 Add empty_parentheses_with_trailing_closure rule
Fixes #885
2016-12-11 02:33:23 -02:00
Marcelo Fabri 2ddf7757d7 Merge branch 'master' into attributes-rule 2016-12-09 14:50:48 -02:00
Marcelo Fabri 205c308100 Merge pull request #904 from marcelofabri/file-header-rule
Add FileHeaderRule
2016-12-09 01:22:58 -02:00
JP Simard 52c025b866 Merge pull request #950 from marcelofabri/redundant-enum
Add RedundantStringEnumValueRule
2016-12-08 14:55:23 -08:00
Marcelo Fabri 13b0cfe623 Fixing inconsistencies 2016-12-08 18:29:57 -02:00
Marcelo Fabri 54d8958386 PR feedback 2016-12-08 18:29:57 -02:00
Marcelo Fabri 4abe67c7d7 Add FileHeaderRule
Fixes #844
2016-12-08 18:29:57 -02:00
Marcelo Fabri a7c58c14d4 Add RedundantStringEnumValueRule
Fixes #946
2016-12-08 18:26:45 -02:00
JP Simard cd632bc106 fix silly typo 2016-12-08 12:13:11 -08:00
JP Simard a57526d639 omit 'self.' references when not ambiguous 2016-12-08 01:25:54 -08:00
Marcelo Fabri 27456823e3 Add number_separator opt-in rule
Fixes #924
2016-12-06 00:17:28 -02:00