Skip to content

changed-files: ignore empty patterns#63

Open
dbarbuzzi wants to merge 1 commit intomainfrom
changed-files-ignored-empty-patterns
Open

changed-files: ignore empty patterns#63
dbarbuzzi wants to merge 1 commit intomainfrom
changed-files-ignored-empty-patterns

Conversation

@dbarbuzzi
Copy link
Contributor

Summary:

Update the changed-files action so that "empty" patterns are ignored (empty/only whitespace). Also, add some logging to help improve clarity.

Test Plan:

# run in a repo where there was one changed file that previously
# wasn't being picked up due to an "empty" exclude pattern
$ export INCLUDE_PATTERN=$(cat <<EOF
^src/
^tests/
^\.github/workflows/test-check-transformers\.yaml$
^MANIFEST\.in$
^setup\.py$
EOF
)
$ export EXCLUDE_PATTERN=$(cat <<EOF
^tests/e2e/
^tests/lmeval/
^tests/examples/
\.md$
EOF
)
$ bash check-files.sh 
CHANGED_FILES: .github/workflows/test-check-transformers.yaml
including: .github/workflows/test-check-transformers.yaml (pattern: ^\.github/workflows/test-check-transformers\.yaml$)
all_changed_files=.github/workflows/test-check-transformers.yaml

Related Issues:

n/a

@dbarbuzzi dbarbuzzi self-assigned this Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants