Skip to content

Add inclusion flags to the apply script #68

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
billwallis merged 1 commit into from
Jun 5, 2026
Merged

Add inclusion flags to the apply script #68

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 23 additions & 2 deletions tools/apply.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
import dataclasses
import logging
import pathlib
import re
import shutil
from collections.abc import Sequence

Expand All @@ -27,6 +28,19 @@ class Context:
target_repo: pathlib.Path | None


def _select_files(
files: list[str],
include: list[str],
exclude: list[str],
) -> list[str]:
return [
file
for file in files
if any(re.match(i, file) for i in include)
and not all(re.match(e, file) for e in exclude)
]


def _apply_template(target: pathlib.Path, context: Context) -> None:
# Right now, this just replaces `python-template` with the name of the
# target repo, but this could be extended to, say, apply Jinja templates
Expand Down Expand Up @@ -56,7 +70,11 @@ def _copy_file_to_target(
_apply_template(destination, context)


def copy_files_to_target(target: pathlib.Path, dry_run: bool) -> int:
def copy_files_to_target(
target: pathlib.Path,
files: list[str],
dry_run: bool,
) -> int:
if not target.exists():
logging.error(f"target {target} does not exist")
return FAILURE
Expand All @@ -65,7 +83,7 @@ def copy_files_to_target(target: pathlib.Path, dry_run: bool) -> int:
return FAILURE

ret = SUCCESS
for file in FILES_TO_COPY:
for file in files:
try:
_copy_file_to_target(
source=ROOT / file,
Expand Down Expand Up @@ -94,6 +112,8 @@ def main(argv: Sequence[str] | None = None) -> int:
action=argparse.BooleanOptionalAction,
default=False,
)
parser.add_argument("--include", nargs="*", default=[".*"])
parser.add_argument("--exclude", nargs="*", default=["^$"])

args = parser.parse_args(argv)
ret = SUCCESS
Expand All @@ -102,6 +122,7 @@ def main(argv: Sequence[str] | None = None) -> int:
for target in args.targets:
ret |= copy_files_to_target(
target=pathlib.Path(target).resolve(),
files=_select_files(FILES_TO_COPY, args.include, args.exclude),
dry_run=args.dry_run,
)

Expand Down