comparison hgext/fix.py @ 43058:808a57a08470

fix: add :enabled sub-config for fixer tools

This allows distributing opt-in fixer tool configurations in .hgrc
files. This may be useful for adding default configs in core, or for
orgranizations that want to provide configs to their users.

Tools are still enabled by default because it would be confusing to add a
config and find that it has no effect until you add enabled=true.

Differential Revision: https://phab.mercurial-scm.org/D6975

43057:c5c502bd1f70

    'linerange': None,
    'pattern': None,
    'priority': 0,
    'metadata': 'false',
    'skipclean': 'true',
}

for key, default in FIXER_ATTRS.items():
    configitem('fix', '.*(:%s)?' % key, default=default, generic=True)

            setattr(fixers[name], pycompat.sysstr('_' + key),
                    attrs.get(key, default))
        fixers[name]._priority = int(fixers[name]._priority)
        fixers[name]._metadata = stringutil.parsebool(fixers[name]._metadata)
        fixers[name]._skipclean = stringutil.parsebool(fixers[name]._skipclean)
        # Don't use a fixer if it has no pattern configured. It would be
        # dangerous to let it affect all files. It would be pointless to let it
        # affect no files. There is no reasonable subset of files to use as the
        # default.
        if fixers[name]._pattern is None:
            ui.warn(
                _('fixer tool has no pattern configuration: %s\n') % (name,))
            del fixers[name]
    return collections.OrderedDict(
        sorted(fixers.items(), key=lambda item: item[1]._priority,
               reverse=True))

43058:808a57a08470

    'linerange': None,
    'pattern': None,
    'priority': 0,
    'metadata': 'false',
    'skipclean': 'true',
    'enabled': 'true',
}

for key, default in FIXER_ATTRS.items():
    configitem('fix', '.*(:%s)?' % key, default=default, generic=True)

            setattr(fixers[name], pycompat.sysstr('_' + key),
                    attrs.get(key, default))
        fixers[name]._priority = int(fixers[name]._priority)
        fixers[name]._metadata = stringutil.parsebool(fixers[name]._metadata)
        fixers[name]._skipclean = stringutil.parsebool(fixers[name]._skipclean)
        fixers[name]._enabled = stringutil.parsebool(fixers[name]._enabled)
        # Don't use a fixer if it has no pattern configured. It would be
        # dangerous to let it affect all files. It would be pointless to let it
        # affect no files. There is no reasonable subset of files to use as the
        # default.
        if fixers[name]._pattern is None:
            ui.warn(
                _('fixer tool has no pattern configuration: %s\n') % (name,))
            del fixers[name]
        elif not fixers[name]._enabled:
            ui.debug('ignoring disabled fixer tool: %s\n' % (name,))
            del fixers[name]
    return collections.OrderedDict(
        sorted(fixers.items(), key=lambda item: item[1]._priority,
               reverse=True))