mirror of
https://github.com/facebook/sapling.git
synced 2024-10-13 02:07:31 +03:00
584656dff3
Summary: Turned on the auto formatter. Ran `arc lint --apply-patches --take BLACK **/*.py`. Then run `arc lint` again so some other autofixers like spellchecker etc. looked at the code base. Manually accept the changes whenever they make sense, or use a workaround (ex. changing "dict()" to "dict constructor") where autofix is false positive. Disabled linters on files that are hard (i18n/polib.py) to fix, or less interesting to fix (hgsubversion tests), or cannot be fixed without breaking OSS build (FBPYTHON4). Conflicted linters (test-check-module-imports.t, part of test-check-code.t, test-check-pyflakes.t) are removed or disabled. Duplicated linters (test-check-pyflakes.t, test-check-pylint.t) are removed. An issue of the auto-formatter is lines are no longer guarnateed to be <= 80 chars. But that seems less important comparing with the benefit auto-formatter provides. As we're here, also remove test-check-py3-compat.t, as it is currently broken if `PYTHON3=/bin/python3` is set. Reviewed By: wez, phillco, simpkins, pkaush, singhsrb Differential Revision: D8173629 fbshipit-source-id: 90e248ae0c5e6eaadbe25520a6ee42d32005621b
65 lines
2.0 KiB
Python
65 lines
2.0 KiB
Python
# hiddenerror.py
|
|
#
|
|
# Copyright 2017 Facebook, Inc.
|
|
#
|
|
# This software may be used and distributed according to the terms of the
|
|
# GNU General Public License version 2 or any later version.
|
|
|
|
"""configurable error messages for accessing hidden changesets
|
|
|
|
Set the following configuration options to customize the error message
|
|
seen when the user attempts to access a hidden changeset::
|
|
|
|
[hiddenerror]
|
|
message = my custom message
|
|
hint = my custom hint
|
|
|
|
The message and hint can contain an optional `{0}` which will be substituted
|
|
with the hash of the hidden changeset.
|
|
"""
|
|
from __future__ import absolute_import
|
|
|
|
from mercurial import context, error, extensions
|
|
from mercurial.i18n import _
|
|
from mercurial.node import short
|
|
|
|
|
|
testedwith = "ships-with-fb-hgext"
|
|
|
|
|
|
def uisetup(ui):
|
|
"""Wrap context.changectx to catch FilteredRepoLookupError."""
|
|
# Get the error messages from the user's configuration and substitute the
|
|
# hash in.
|
|
msgfmt, hintfmt = _getstrings(ui)
|
|
|
|
def _filterederror(orig, repo, rev):
|
|
# If the number is beyond the changelog, it's a short hash that
|
|
# just happened to be a number.
|
|
intrev = None
|
|
try:
|
|
intrev = int(rev)
|
|
except ValueError:
|
|
pass
|
|
if intrev is not None and intrev < len(repo):
|
|
node = repo.unfiltered()[rev].node()
|
|
shorthash = short(node)
|
|
msg = msgfmt.format(shorthash)
|
|
hint = hintfmt and hintfmt.format(shorthash)
|
|
return error.FilteredRepoLookupError(msg, hint=hint)
|
|
return orig(repo, rev)
|
|
|
|
extensions.wrapfunction(context, "_filterederror", _filterederror)
|
|
|
|
|
|
def _getstrings(ui):
|
|
"""Lood the error messages to show when the user tries to access a
|
|
hidden commit from the user's configuration file. Fall back to
|
|
default messages if nothing is configured.
|
|
"""
|
|
msg = ui.config("hiddenerror", "message")
|
|
hint = ui.config("hiddenerror", "hint")
|
|
if not msg:
|
|
msg = _("hidden changeset {0}")
|
|
return msg, hint
|