Merge pull request #16 from Workiva/suppress-output-error-option

Add new option to suppress output errors

Author: rmconsole3-wf

Dockerfile

@@ -10,7 +10,7 @@ RUN yum install -y \
 
 RUN curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.34.0/install.sh | bash && \
     . ~/.nvm/nvm.sh && \
-    nvm install node && \
+    nvm install 17 && \
     npm install yarn -g && \
     yarn install  && \
     yarn compile && \

over-react-format-on-save-1.0.0.vsix over-react-format-on-save-1.0.1.vsixover-react-format-on-save-1.0.0.vsix renamed to over-react-format-on-save-1.0.1.vsix

@@ -2,7 +2,7 @@
 	"name": "over-react-format-on-save",
 	"displayName": "OverReact Format on Save",
 	"description": "A Dart project formatting tool that enables formatting on file save. Attempts to use OverReact Format but defaults to dartfmt.",
-	"version": "1.0.0",
+	"version": "1.0.1",
 	"publisher": "Workiva",
 	"repository": "https://github.com/Workiva/vs-code-format-on-save/",
 	"engines": {
@@ -12,7 +12,7 @@
 		"Other"
 	],
 	"activationEvents": [
-		"*"
+		"onLanguage:dart"
 	],
 	"main": "./out/extension.js",
 	"contributes": {
@@ -47,6 +47,12 @@
 					"type": "boolean",
 					"description": "Allows the extension to scan for pubspec.yaml files that are nested somewhere in the project (defaulting to the project root) when the formatter runs.\n\nSee the extension details for more information.",
 					"default": false
+				},
+				"overReact.formatOnSave.showOutputOnError": {
+					"Title": "Show output console when error occurs in formatter",
+					"type": "boolean",
+					"description": "By default the extension will show the output console when an error occurs in the formatter. This can be disabled.",
+					"default": true
 				}
 			}
 		}

package.json

@@ -186,11 +186,14 @@ class RunFormatOnSave {
 
 		child.on('exit', (e) => {
 			if (e === 0) {
-				this.showStatusMessage('Formatting succeeded');
-			}
+				this.showStatusMessage('OverReact formatting succeeded');
+			} else {
+				this.showStatusMessage('OverReact formatting failed');
 
-			if (e !== 0) {
-				this.channel.show(true);
+				const shouldShowOutputOnError = this.config.get<Boolean>('showOutputOnError');
+				if (shouldShowOutputOnError) {
+					this.channel.show(true);
+				}
 			}
 		});
 	}