This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	martinvonz	D7086 fix: remove a never-true check for unset pattern in Fixer.affects()
Closed	martinvonz	D7085 fix: warn when a fixer doesn't have a configured command
Closed	martinvonz	D7084 fix: make Fixer initialization more explicit for clarity
Closed	martinvonz	D7083 fix: don't pass in default value when looking up config
Closed	martinvonz	D7082 fix: fix registration of config item defaults

Diff 17144

	self._pattern = pattern			self._pattern = pattern
	self._linerange = linerange			self._linerange = linerange
	self._priority = priority			self._priority = priority
	self._metadata = metadata			self._metadata = metadata
	self._skipclean = skipclean			self._skipclean = skipclean

	def affects(self, opts, fixctx, path):			def affects(self, opts, fixctx, path):
	"""Should this fixer run on the file at the given path and context?"""			"""Should this fixer run on the file at the given path and context?"""
	return self._pattern is not None and scmutil.match(			return scmutil.match(fixctx, [self._pattern], opts)(path)
	fixctx, [self._pattern], opts
	)(path)

	def shouldoutputmetadata(self):			def shouldoutputmetadata(self):
	"""Should the stdout of this fixer start with JSON and a null byte?"""			"""Should the stdout of this fixer start with JSON and a null byte?"""
	return self._metadata			return self._metadata

	def command(self, ui, path, ranges):			def command(self, ui, path, ranges):
	"""A shell command to use to invoke this fixer on the given file/lines			"""A shell command to use to invoke this fixer on the given file/lines