Commit Graph

1143 Commits

Author SHA1 Message Date
rakaramos da071774e0 Add UnusedOptionalBinding rule 2017-01-05 21:13:03 -02:00
JP Simard dc82a5473c Merge pull request #1112 from VojtaStavik/correctable-nimble-operator-rule
Make NimbleOperatorRule correctable
2017-01-05 07:08:23 -08:00
Vojta Stavik f4bc2288e7 Remove labeled loops, better formatting 2017-01-05 11:20:20 +01:00
Aaron McTavish 00b8e79318 Add max_empty_lines config to vertical_whitespace
Update `vertical_whitespace` rule to allow configuration of the number
of consecutive empty lines before a violation using `max_empty_lines`.
The default value is still 1 line.
2017-01-05 10:05:03 +00:00
Marcelo Fabri 6d4dd49902 PR feedback 2017-01-05 00:41:11 -02:00
Marcelo Fabri 5ca89f270e Refactor split to use String.components(separatedBy:) 2017-01-05 00:41:11 -02:00
Marcelo Fabri 5a22c71ccb PR feedback 2017-01-05 00:41:11 -02:00
Marcelo Fabri fdfb424eff Add support to generic typealias 2017-01-05 00:41:11 -02:00
Marcelo Fabri 4fb8fd84c0 Support generic enum 2017-01-05 00:41:11 -02:00
Marcelo Fabri 0c0780188e Applying hack on Linux 2017-01-05 00:41:11 -02:00
Marcelo Fabri dd59f4e9f9 Add generic_type_name rule
Fixes #51
2017-01-05 00:41:11 -02:00
Marcelo Fabri 6e97d760bc PR feedback 2017-01-04 21:30:59 -02:00
Marcelo Fabri 6fc77e0b9b Ignore closure declarations 2017-01-04 18:21:33 -02:00
Marcelo Fabri 0e65f89620 Alternative implemention that works on Linux 2017-01-04 18:21:33 -02:00
Marcelo Fabri c5db646115 Add large_tuple rule
Fixes #1065
2017-01-04 18:21:33 -02:00
Marcelo Fabri ccdff37f1d PR feedback 2017-01-04 15:35:07 -02:00
Marcelo Fabri 66403ec5fa Handle other protocols 2017-01-04 15:34:19 -02:00
Marcelo Fabri c381720d3e Add expressible_by_array_literal_init rule
Fixes #1096
2017-01-04 15:34:19 -02:00
Vojta Stavik 10dc5afa99 Add missing Foundation import 2017-01-04 17:16:56 +01:00
Vojta Stavik ba3b4a1728 Remove redundant label 2017-01-04 16:59:11 +01:00
Vojta Stavik d8179346ed Improve NimbleOperatorRule corrections 2017-01-04 16:54:13 +01:00
JP Simard cce3505f1b pass function reference rather than create a small closure 2017-01-03 23:30:55 -08:00
JP Simard 6e994776df use guard instead of nested if let 2017-01-03 23:17:28 -08:00
Marcelo Fabri 12a66d4fe1 Remove handleAlias function 2017-01-03 18:04:37 -02:00
Marcelo Fabri 3b682fb116 Remove support to non-deprecated aliases 2017-01-03 16:14:34 -02:00
Vojta Stavik d60f11c6a7 Make NimbleOperatorRule correctable 2017-01-03 14:02:31 +01:00
Marcelo Fabri 3c9d16509b Fix unit tests 2017-01-02 18:27:28 -02:00
JP Simard 4519f463d8 extract Linter.getStyleViolations' parallelFlatMap into Rule.performLint 2017-01-02 12:01:33 -08:00
Marcelo Fabri d9eda655bf PR feedback 2017-01-02 17:49:54 -02:00
Marcelo Fabri f5569986a3 Only dispatching if there are identifiers 2017-01-02 16:26:16 -02:00
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 95a468a723 Merge branch 'master' into jp-parallelize
* master:
  use first(where:) instead of for loop in RulesCommand.swift
  remove unnecessary imports from AutoCorrectCommand.swift
  Typo in changelog
  Remove unnecessary import
  Avoid duplicating default value
  Update number_separator configuration in SwiftLint
  number_separator can be configured with a min length
  Enable more opt-in rules in SwiftLint
  Fix out of range exception in AttributesRule
  Fix trailing_comma crash
2017-01-02 09:55:32 -08:00
JP Simard 8906da640c perform Cache factory while locked 2017-01-02 09:52:01 -08:00
JP Simard 53d0cf6975 fix incorrect flatMap usage to use map instead 2017-01-02 09:47:48 -08:00
Marcelo Fabri 5282134fec Remove unnecessary import 2017-01-02 03:25:04 -02:00
Marcelo Fabri d44bf896b9 Avoid duplicating default value 2017-01-02 02:55:41 -02:00
Marcelo Fabri ef8bc05b89 Update number_separator configuration in SwiftLint 2017-01-02 02:51:12 -02:00
Marcelo Fabri 09a251a8f6 number_separator can be configured with a min length
Fixes #1109
2017-01-02 02:50:04 -02:00
Marcelo Fabri ca5914816f Enable more opt-in rules in SwiftLint 2017-01-02 01:59:34 -02:00
Marcelo Fabri f9a639715f Fix out of range exception in AttributesRule
Fixes #1105
2017-01-01 17:54:31 -02:00
Marcelo Fabri 1a9d2ef7e6 Fix trailing_comma crash
Fixes #921
2017-01-01 01:14:19 -02:00
JP Simard 253e1f4f86 make recording benchmarks thread-safe 2016-12-31 17:24:43 -08:00
JP Simard 8774a921a6 use DispatchQueue.concurrentPerform rather than rolling our own
also remove parallelForEach from SwiftLintFramework's public interface
since calling the GCD API is almost as concise and avoids polluting the
public API.

Finally, also remove the "fast" path for parallelMap since GCD should
do the reasonable thing for us here.
2016-12-31 17:06:02 -08:00
JP Simard d82ca686d2 parallelize getting violations for a single file 2016-12-30 23:08:26 -08:00