Add fixable status to verbose and github formatters

lib/formatters/__tests__/githubFormatter.test.js

@@ -34,11 +34,11 @@ test('githubFormatter', () => {
 	const returnValue = {
 		ruleMetadata: {
 			foo: { url: 'https://stylelint.io/rules/foo' },
-			bar: {},
+			bar: { fixable: true },
 		},
 	};
 
 	expect(githubFormatter(results, returnValue))
 		.toBe(`::error file=path/to/file.css,line=1,col=2,endLine=1,endColumn=5,title=Stylelint problem::Unexpected "foo" (foo) - https://stylelint.io/rules/foo
-::warning file=a.css,line=10,col=20,title=Stylelint problem::Unexpected "bar" (bar)`);
+::warning file=a.css,line=10,col=20,title=Stylelint problem::Unexpected "bar" (bar) [maybe fixable]`);
 });

lib/formatters/__tests__/prepareFormatterOutput.js

@@ -9,8 +9,8 @@ symbolConversions.set('✔', '√');
 symbolConversions.set('⚠', '‼');
 symbolConversions.set('✖', '×');
 
-module.exports = function (results, formatter) {
-	const returnValue = {
+module.exports = function (results, formatter, returnValue) {
+	returnValue = returnValue || {
 		ruleMetadata: {},
 	};
 	let output = stripAnsi(formatter(results, returnValue)).trim();

lib/formatters/__tests__/verboseFormatter.test.js

@@ -340,17 +340,15 @@ describe('verboseFormatter', () => {
 				],
 				deprecations: [],
 				invalidOptionWarnings: [],
-				_postcssResult: {
-					stylelint: {
-						ruleMetadata: {
-							'no-foo': { url: 'https://stylelint.io' },
-						},
-					},
-				},
 			},
 		];
+		const returnValue = {
+			ruleMetadata: {
+				'no-foo': { url: 'https://stylelint.io', fixable: true },
+			},
+		};
 
-		const output = prepareFormatterOutput(results, verboseFormatter);
+		const output = prepareFormatterOutput(results, verboseFormatter, returnValue);
 
 		expect(output).toBe(stripIndent`
 			file.css
@@ -362,6 +360,6 @@ describe('verboseFormatter', () => {
 			 file.css
 
 			1 error found
-			 no-foo: 1`);
+			 no-foo: 1 (maybe fixable)`);
 	});
 });

lib/formatters/githubFormatter.js

@@ -30,7 +30,8 @@ module.exports = function githubFormatter(results, returnValue) {
 function buildMessage(msg, metadata) {
 	if (!metadata) return msg;
 
-	if (!metadata.url) return msg;
+	const url = metadata.url ? ` - ${metadata.url}` : '';
+	const fixable = metadata.fixable ? ' [maybe fixable]' : '';
 
-	return `${msg} - ${metadata.url}`;
+	return `${msg}${fixable}${url}`;
 }

lib/formatters/verboseFormatter.js

@@ -71,7 +71,10 @@ module.exports = function verboseFormatter(results, returnValue) {
 			const metadata = returnValue.ruleMetadata;
 
 			for (const [rule, list] of Object.entries(problemsByRule)) {
-				output += dim(` ${ruleLink(rule, metadata[rule])}: ${list.length}\n`);
+				const meta = metadata[rule];
+				const fixable = meta && meta.fixable ? ' (maybe fixable)' : '';
+
+				output += dim(` ${ruleLink(rule, meta)}: ${list.length}${fixable}\n`);
 			}
 		};
 

scripts/visual-config.json

@@ -9,6 +9,7 @@
 				"message": "Expected custom property name to be kebab-case",
 				"severity": "warning"
 			}
-		]
+		],
+		"import-notation": "string"
 	}
 }

scripts/visual.css

@@ -1,3 +1,5 @@
+@import url("foo.css");
+
 .foo {
   color:#4f;
 }