| pulkit |
| hg-reviewers |
The defaults are now registered correctly so we don't need to pass in
the default value when we look up a config value.
| Lint Skipped |
| Unit Tests Skipped |
| Path | Packages | |||
|---|---|---|---|---|
| M | hgext/fix.py (4 lines) |
| Commit | Parents | Author | Summary | Date |
|---|---|---|---|---|
| 790cbb9cb77a | 0082726490ae | Martin von Zweigbergk | Oct 12 2019, 2:16 PM |
| Status | Author | Revision | |
|---|---|---|---|
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz |
| """Returns a map of configured fixer tools indexed by their names | """Returns a map of configured fixer tools indexed by their names | ||||
| Each value is a Fixer object with methods that implement the behavior of the | Each value is a Fixer object with methods that implement the behavior of the | ||||
| fixer's config suboptions. Does not validate the config values. | fixer's config suboptions. Does not validate the config values. | ||||
| """ | """ | ||||
| fixers = {} | fixers = {} | ||||
| for name in fixernames(ui): | for name in fixernames(ui): | ||||
| fixers[name] = Fixer() | fixers[name] = Fixer() | ||||
| for key, default in FIXER_ATTRS.items(): | for key in FIXER_ATTRS: | ||||
| setattr( | setattr( | ||||
| fixers[name], | fixers[name], | ||||
| pycompat.sysstr(b'_' + key), | pycompat.sysstr(b'_' + key), | ||||
| ui.config(b'fix', name + b':' + key, default), | ui.config(b'fix', name + b':' + key), | ||||
| ) | ) | ||||
| fixers[name]._priority = int(fixers[name]._priority) | fixers[name]._priority = int(fixers[name]._priority) | ||||
| fixers[name]._metadata = stringutil.parsebool(fixers[name]._metadata) | fixers[name]._metadata = stringutil.parsebool(fixers[name]._metadata) | ||||
| fixers[name]._skipclean = stringutil.parsebool(fixers[name]._skipclean) | fixers[name]._skipclean = stringutil.parsebool(fixers[name]._skipclean) | ||||
| fixers[name]._enabled = stringutil.parsebool(fixers[name]._enabled) | fixers[name]._enabled = stringutil.parsebool(fixers[name]._enabled) | ||||
| # Don't use a fixer if it has no pattern configured. It would be | # 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 | # 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 | # affect no files. There is no reasonable subset of files to use as the | ||||