mirror of
https://github.com/digital-asset/daml.git
synced 2024-09-19 08:48:21 +03:00
ci/cron: fix Wednesday (#14297)
There's a small, subtle bug that made its way into #14287: apparently `/dev/stdout` is not defined on CI. On the bright side, we don't actually need the `out` parameter, so removing it seems like the easiest fix here. CHANGELOG_BEGIN CHANGELOG_END
This commit is contained in:
parent
5c026163a9
commit
6cdb68daca
@ -32,12 +32,11 @@ steps:
|
|||||||
fi
|
fi
|
||||||
}
|
}
|
||||||
open_pr() {
|
open_pr() {
|
||||||
local branch title body out pr_number header output
|
local branch title body pr_number header output
|
||||||
branch="$1"
|
branch="$1"
|
||||||
title="$2"
|
title="$2"
|
||||||
body="${3:-}"
|
body="${3:-}"
|
||||||
out="${4:-/dev/stdout}"
|
pr_number="${4:-}"
|
||||||
pr_number="${5:-}"
|
|
||||||
header=$(mktemp)
|
header=$(mktemp)
|
||||||
output=$(mktemp)
|
output=$(mktemp)
|
||||||
|
|
||||||
@ -61,9 +60,7 @@ steps:
|
|||||||
--output "$output" \
|
--output "$output" \
|
||||||
-d @- \
|
-d @- \
|
||||||
https://api.github.com/repos/digital-asset/daml/pulls
|
https://api.github.com/repos/digital-asset/daml/pulls
|
||||||
if [ -n "$out" ]; then
|
cat "$header" "$output"
|
||||||
cat "$header" "$output" > "$out"
|
|
||||||
fi
|
|
||||||
if [ -n "$pr_number" ]; then
|
if [ -n "$pr_number" ]; then
|
||||||
jq '.number' "$output" > "$pr_number"
|
jq '.number' "$output" > "$pr_number"
|
||||||
fi
|
fi
|
||||||
|
@ -48,6 +48,6 @@ jobs:
|
|||||||
|
|
||||||
rotate
|
rotate
|
||||||
git add .
|
git add .
|
||||||
open_pr "$BRANCH" "$TITLE" "$BODY" "/dev/null" "$PR_NUMBER"
|
open_pr "$BRANCH" "$TITLE" "$BODY" "$PR_NUMBER"
|
||||||
trigger_azure $(System.AccessToken) "PRs" --branch $BRANCH
|
trigger_azure $(System.AccessToken) "PRs" --branch $BRANCH
|
||||||
request_pr_review $(cat "$PR_NUMBER") "$NEXT_GH"
|
request_pr_review $(cat "$PR_NUMBER") "$NEXT_GH"
|
||||||
|
Loading…
Reference in New Issue
Block a user