Differential D12198 Diff 29316 src/lint/linter/ArcanistESLintLinter.php

Changeset View

Standalone View

src/lint/linter/ArcanistESLintLinter.php

This file was added.

				<?php

				final class ArcanistESLintLinter extends ArcanistExternalLinter {

				public function getInfoName() {
				return 'ESLint';
				}

				public function getInfoURI() {
				return 'https://www.eslint.org';
				}

				public function getInfoDescription() {
				return pht('ESLint is a linter for JavaScript source files.');
				}
				michaeljs1990AuthorUnsubmitted Not Done Inline Actions This is the only way I see presently to set a min version. However as noted above the user can potential mess this up by setting a lower version in .arclint or .arcconfig and I have no way to stop this presently. michaeljs1990: This is the only way I see presently to set a min version. However as noted above the user can…

				public function getLinterName() {
				return 'ESLint';
				}
				joshuaspenceUnsubmitted Done Inline Actions Linter names are typically uppercase, consider changing this to `ESLINT`. joshuaspence: Linter names are typically uppercase, consider changing this to `ESLINT`.

				aviveyUnsubmitted Not Done Inline Actions This was actually changed since - see T9353. The idea is that the one-liner output from `arc linters` should be useful for someone who never used eslint/javascript. avivey: This was actually changed since - see T9353. The idea is that the one-liner output from `arc…
				public function getLinterConfigurationName() {
				return 'eslint';
				}

				public function getDefaultBinary() {
				return 'eslint';
				}

				public function getInstallInstructions() {
				return pht('Install ESLint using `npm install -g eslint`.');
				}
				joshuaspenceUnsubmitted Done Inline Actions Technically this should be written as `pht('Install ESLint using` `%s`.', 'npm install -g eslint') because `npm install -g eslint` isn't really translatable. joshuaspence: Technically this should be written as `pht('Install ESLint using ```%s```.', 'npm install -g…
				bspoonUnsubmitted Not Done Inline Actions Hi! I tried using this linter, and configured it to use the eslint binary relative to my project (`node_modules/.bin/eslint`), and it wouldn't report the version (because I don't have eslint installed globally). Maybe the line should be this, instead? list($output) = execx('%C --version', $this->getExecutableCommand()); bspoon: Hi! I tried using this linter, and configured it to use the eslint binary relative to my…
				michaeljs1990AuthorUnsubmitted Not Done Inline Actions You are correct, sorry I missed that. Will push up a change for it shortly. michaeljs1990: You are correct, sorry I missed that. Will push up a change for it shortly.

				protected function canCustomizeLintSeverities() {
				return true;
				}

				joshuaspenceUnsubmitted Done Inline Actions This is the default value, just remove this method. joshuaspence: This is the default value, just remove this method.
				protected function parseLinterOutput($path, $err, $stdout, $stderr) {
				$lines = phutil_split_lines($stdout, false);

				$messages = array();
				foreach ($lines as $line) {
				// Clean up nasty ESLint output
				$clean_line = $output = preg_replace('!\s+!', ' ', $line);
				$parts = explode(' ', ltrim($clean_line));

				if (isset($parts[1]) &&
				($parts[1] === 'error' \|\| $parts[1] === 'warning')) {
				$severity = $parts[1] === 'error' ?
				ArcanistLintSeverity::SEVERITY_ERROR :
				ArcanistLintSeverity::SEVERITY_WARNING;

				$message = new ArcanistLintMessage();
				$message->setPath($path);
				$message->setLine($parts[0]);
				$message->setCode($this->getLinterName());
				$message->setDescription(implode(' ', $parts));
				$message->setSeverity($severity);

				$messages[] = $message;
				}
				}

				return $messages;
				}

				}