Skip to content

Commit a589654

Browse files
webknjazhugovk
authored andcommitted
Use CSV-separated outputs @ get-changed-files @ CI
This is necessary because paths with whitespaces tend to crash said action[[1]][[2]][[3]]. Also, we don't need to use JSON as it's harder to parse while the value isn't used except for the emptiness check. The change fixes [[4]] [1]: https://github.com/Ana06/get-changed-files#get-all-changed-files-as-space-delimited [2]: python#103914 (comment) [3]: python#103914 (comment) [4]: python#103914 (comment)
1 parent a96bfc5 commit a589654

2 files changed

Lines changed: 2 additions & 0 deletions

File tree

.github/workflows/build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,7 @@ jobs:
8989
Doc/**
9090
Misc/**
9191
.github/workflows/reusable-docs.yml
92+
format: csv # unbreaks paths /w whitespaces; json is harder to parse
9293
- name: Check for docs changes
9394
if: >-
9495
github.event_name == 'pull_request'

.github/workflows/reusable-docs.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ jobs:
4040
uses: Ana06/get-changed-files@v2.2.0
4141
with:
4242
filter: "Doc/**"
43+
format: csv # unbreaks paths /w whitespaces; json is harder to parse
4344
- name: 'Build changed files in nit-picky mode'
4445
if: github.event_name == 'pull_request'
4546
continue-on-error: true

0 commit comments

Comments
 (0)