Skip to content
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

Jump to bottom

fix(regression): invalid json output. #930

Merged
merged 5 commits into from Jan 5, 2023
Merged

fix(regression): invalid json output. #930

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
042d993
fix(regression): invalid json output for changed files within a direc…
jackton1 Jan 5, 2023
74baacc
Update test.yml
jackton1 Jan 5, 2023
f739384
Update get-changed-paths.sh
jackton1 Jan 5, 2023
f939c41
Fixed json_output function
jackton1 Jan 5, 2023
d662faa
Merge branch 'main' into fix/invalid-json-output-within-a-directory
repo-ranger[bot] Jan 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 0 additions & 1 deletion .github/workflows/test.yml
View file
Open in desktop
Expand Up @@ -3,7 +3,6 @@ name: CI
on:
push:
branches:
- main
- "**"
pull_request:
types:
Expand Down
4 changes: 2 additions & 2 deletions get-changed-paths.sh
View file
Open in desktop
Expand Up @@ -52,13 +52,13 @@ function get_dirname_max_depth() {
}

function json_output() {
JQ_ARGS="-R"
JQ_ARGS="-sR"
if [[ "$INPUT_JSON_RAW_FORMAT" == "true" ]]; then
JQ_ARGS="$JQ_ARGS -r"
fi

# shellcheck disable=SC2086
jq $JQ_ARGS 'split("\n") | @json' | tr -s /
jq $JQ_ARGS 'split("\n") | map(select(. != "")) | @json' | sed -r 's/^"|"$//g' | tr -s /
}

function get_diff() {
Expand Down
2 changes: 1 addition & 1 deletion test/test/test.txt
View file
Open in desktop
@@ -1 +1 @@
This is a test file
This is a test file.