diff options
author | Philipp Albrecht <palbrecht@mailbox.org> | 2023-06-30 11:03:19 +0200 |
---|---|---|
committer | Philipp Albrecht <palbrecht@mailbox.org> | 2023-06-30 11:03:19 +0200 |
commit | 6f727664a9d8945e1f536b69ea0f3b0aae9e29e0 (patch) | |
tree | 959558977fbf5f99e406d243ee835727c89e1696 | |
parent | d9e8b638bf6d20ff6d2179db7c105385c8ddad1d (diff) | |
download | qutebrowser-6f727664a9d8945e1f536b69ea0f3b0aae9e29e0.tar.gz qutebrowser-6f727664a9d8945e1f536b69ea0f3b0aae9e29e0.zip |
Add misc check to prevent vim modelines
We deprecated vim modelines. The introduced misc check will keep old pull requests from
re-introducing vim modelines.
-rw-r--r-- | scripts/dev/misc_checks.py | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/scripts/dev/misc_checks.py b/scripts/dev/misc_checks.py index 82c3d48c1..a1df9e51d 100644 --- a/scripts/dev/misc_checks.py +++ b/scripts/dev/misc_checks.py @@ -397,6 +397,25 @@ def check_userscript_shebangs(_args: argparse.Namespace) -> bool: return ok +def check_vim_modelines(args: argparse.Namespace) -> bool: + """Check that we're not using vim modelines.""" + ok = True + try: + for path in _get_files(verbose=args.verbose): + with tokenize.open(str(path)) as f: + for num, line in enumerate(f, start=1): + if not line.startswith("# vim:"): + continue + print(f"{path}:{num}: Remove vim modeline " + "(deprecated in favor of .editorconfig)") + ok = False + except Exception: + traceback.print_exc() + ok = False + + return ok + + def main() -> int: checkers = { 'git': check_git, @@ -406,6 +425,7 @@ def main() -> int: 'userscript-descriptions': check_userscripts_descriptions, 'userscript-shebangs': check_userscript_shebangs, 'changelog-urls': check_changelog_urls, + 'vim-modelines': check_vim_modelines, } parser = argparse.ArgumentParser() |