This is an archive of the discontinued Mercurial Phabricator instance.

Status	Author	Revision
Closed	durin42	D6997 contrib: fix check-code to be able to detect missing _() with bytestrings
Closed	durin42	D6996 cleanup: mark some ui.(status\|note\|warn\|write) calls as not needing i18n
Closed	indygreg	D6994 ui: define (write\|status\|warn\|note)noi18n aliases
Closed	durin42	D6973 mangler: stop rewriting string constants to be bytes literals
Closed	durin42	D6972 formatting: byteify all mercurial/ and hgext/ string literals
Closed	durin42	D6971 formatting: blacken the codebase
Closed	durin42	D6970 transaction: re-wrap line to avoid a black bug
Closed	durin42	D6969 discovery: re-wrap expression to avoid a black bug
Closed	durin42	D6342 style: run a patched black on a subset of mercurial
Closed	durin42	D6974 examples: include a sample of how to use black with fix
Closed	durin42	D6993 black: add a pyproject.toml that contains our black settings
Closed	durin42	D6979 contrib: stop whining about whitespace around =
Closed	durin42	D6978 contrib: black manages whitespace after Python keywords now
Closed	durin42	D6977 contrib: allow trailing commas in lists
Closed	durin42	D6976 contrib: remove check-code line length limit
Closed	durin42	D6985 statprof: drop python #! and exec bit we don't use
Closed	durin42	D6984 cvsps: switch a file open to a with statement

Diff 16828

	(r'\.has_key\b', "dict.has_key is not available in Python 3+"),			(r'\.has_key\b', "dict.has_key is not available in Python 3+"),
	(r'\s<>\s', '<> operator is not available in Python 3+, use !='),			(r'\s<>\s', '<> operator is not available in Python 3+, use !='),
	(r'^\s*\t', "don't use tabs"),			(r'^\s*\t', "don't use tabs"),
	(r'\S;\s*\n', "semicolon"),			(r'\S;\s*\n', "semicolon"),
	(r'[^_]_\([ \t\n](?:"[^"]+"[ \t\n+])+%', "don't use % inside _()"),			(r'[^_]_\([ \t\n](?:"[^"]+"[ \t\n+])+%', "don't use % inside _()"),
	(r"[^_]_\([ \t\n](?:'[^']+'[ \t\n+])+%", "don't use % inside _()"),			(r"[^_]_\([ \t\n](?:'[^']+'[ \t\n+])+%", "don't use % inside _()"),
	(r'(\w\|\)),\w', "missing whitespace after ,"),			(r'(\w\|\)),\w', "missing whitespace after ,"),
	(r'(\w\|\))[+/*\-<>]\w', "missing whitespace in expression"),			(r'(\w\|\))[+/*\-<>]\w', "missing whitespace in expression"),
	(r'^\s+(\w\|\.)+=\w[^,()\n]*$', "missing whitespace in assignment"),
	(r'\w\s=\s\s+\w', "gratuitous whitespace after ="),			(r'\w\s=\s\s+\w', "gratuitous whitespace after ="),
	((			((
	# a line ending with a colon, potentially with trailing comments			# a line ending with a colon, potentially with trailing comments
	r':([ \t]#[^\n])?\n'			r':([ \t]#[^\n])?\n'
	# one that is not a pass and not only a comment			# one that is not a pass and not only a comment
	r'(?P<indent>[ \t]+)[^#][^\n]+\n'			r'(?P<indent>[ \t]+)[^#][^\n]+\n'
	# more lines at the same indent level			# more lines at the same indent level
	r'((?P=indent)[^\n]+\n)*'			r'((?P=indent)[^\n]+\n)*'