Commit Graph

231 Commits

Author SHA1 Message Date
Marcelo Fabri b3cd5008c0 Make LinterCache final 2016-12-31 01:42:18 -02:00
Marcelo Fabri c7461e0bbb Supporting CLI and configuration options 2016-12-31 00:26:05 -02:00
Marcelo Fabri 3d376241be Moving cache usage to the linter 2016-12-31 00:26:05 -02:00
Marcelo Fabri 34af12a2de Temporary fix for getting version on SPM 2016-12-31 00:24:54 -02:00
Marcelo Fabri 77e4249a3d Improved cache structure 2016-12-31 00:24:54 -02:00
Marcelo Fabri a47385a64f Ignore cache if configuration has changed 2016-12-31 00:24:54 -02:00
Marcelo Fabri b1f62e2b63 Add some tests 2016-12-31 00:24:54 -02:00
Marcelo Fabri f1e33760f0 Fixing compile isse 2016-12-31 00:24:54 -02:00
Marcelo Fabri a92a24c598 Ignore cache when versions are different 2016-12-31 00:24:54 -02:00
Marcelo Fabri c698d021dc Initial cache implementation 2016-12-31 00:24:54 -02: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