Skip to content

Commit

Permalink
get-ci-matrix.py: ignore IMPLS.yml and Makefile.impls
Browse files Browse the repository at this point in the history
This isn't a perfect solution because if somebody pushes a new
implementation and then later pushes the updates to these files, it
probably won't trigger any implementations. However, the workflow can
always be manually triggered for a specific implementation and branch so
that's probably okay. At least this way, the addition of a new
implementation won't always trigger every other implementation to be
tested.
  • Loading branch information
kanaka committed Aug 8, 2024
1 parent 205cf51 commit f5d6d0e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions get-ci-matrix.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
import yaml

IMPLS_FILE = "IMPLS.yml"
RE_IGNORE = re.compile(r'(^LICENSE$|^README.md$|^docs/|^process/)')
RE_IGNORE = re.compile(r'(^LICENSE$|^README.md$|^docs/|^process/|^IMPLS.yml$|^Makefile.impls$)')
RE_IMPL = re.compile(r'^impls/(?!lib|tests)([^/]*)/')

OVERRIDE_IMPLS = os.environ.get('OVERRIDE_IMPLS', '').split()
Expand All @@ -23,7 +23,7 @@ def impl_text(impl):
return s

all_changes = sys.argv[1:]
# code changes that are not just to docs
# code changes that are not just to docs or implementation lists
code_changes = set([c for c in all_changes if not RE_IGNORE.search(c)])
# actual changes to implementations
impl_changes = set([c for c in all_changes if RE_IMPL.search(c)])
Expand Down

0 comments on commit f5d6d0e

Please sign in to comment.