Merge pull request #608 from BrianHenryIE/default-all-targets-fix-483

Default to all targets when plugin `--target` parameter missing. Fix #483

Author: allevato

Plugins/FormatPlugin/plugin.swift

@@ -38,7 +38,7 @@ extension FormatPlugin: CommandPlugin {
 
     var argExtractor = ArgumentExtractor(arguments)
     let targetNames = argExtractor.extractOption(named: "target")
-    let targetsToFormat = try context.package.targets(named: targetNames)
+    let targetsToFormat = targetNames.isEmpty ? context.package.targets : try context.package.targets(named: targetNames)
 
     let configurationFilePath = argExtractor.extractOption(named: "configuration").first
 

Plugins/LintPlugin/plugin.swift

@@ -39,8 +39,8 @@ extension LintPlugin: CommandPlugin {
     // Extract the arguments that specify what targets to format.
     var argExtractor = ArgumentExtractor(arguments)
     let targetNames = argExtractor.extractOption(named: "target")
-    let targetsToFormat = try context.package.targets(named: targetNames)
 
+    let targetsToFormat = targetNames.isEmpty ? context.package.targets : try context.package.targets(named: targetNames)
     let configurationFilePath = argExtractor.extractOption(named: "configuration").first
 
     let sourceCodeTargets = targetsToFormat.compactMap { $0 as? SourceModuleTarget }