Skip to content

Commit

Permalink
Merge pull request #48102 from liggitt/optional-doc-generation
Browse files Browse the repository at this point in the history
Automatic merge from submit-queue (batch tested with PRs 49043, 49001, 49057, 49066, 48102)

Make doc generation on cherry-picks optional

Follow up of #46993, xref #44533

Most cherry-picks don't require doc changes, and doing it unconditionally actually breaks picking to the release-1.6 branch
  • Loading branch information
Kubernetes Submit Queue committed Jul 18, 2017
2 parents c3fa13a + 6967ae7 commit 0d88afa
Showing 1 changed file with 11 additions and 5 deletions.
16 changes: 11 additions & 5 deletions hack/cherry_pick_pull.sh
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ cd "${KUBE_ROOT}"
declare -r STARTINGBRANCH=$(git symbolic-ref --short HEAD)
declare -r REBASEMAGIC="${KUBE_ROOT}/.git/rebase-apply"
DRY_RUN=${DRY_RUN:-""}
REGENERATE_DOCS=${REGENERATE_DOCS:-""}
UPSTREAM_REMOTE=${UPSTREAM_REMOTE:-upstream}
FORK_REMOTE=${FORK_REMOTE:-origin}

Expand All @@ -54,6 +55,9 @@ if [[ "$#" -lt 2 ]]; then
echo " This is useful for creating patches to a release branch without making a PR."
echo " When DRY_RUN is set the script will leave you in a branch containing the commits you cherry-picked."
echo
echo " Set the REGENERATE_DOCS environment var to regenerate documentation for the target branch after picking the specified commits."
echo " This is useful when picking commits containing changes to API documentation."
echo
echo " Set UPSTREAM_REMOTE (default: upstream) and FORK_REMOTE (default: origin)"
echo " To override the default remote names to what you have locally."
exit 2
Expand Down Expand Up @@ -184,12 +188,14 @@ done
gitamcleanup=false

# Re-generate docs (if needed)
echo
echo "Regenerating docs..."
if ! hack/generate-docs.sh; then
if [[ -n "${REGENERATE_DOCS}" ]]; then
echo
echo "hack/generate-docs.sh FAILED to complete."
exit 1
echo "Regenerating docs..."
if ! hack/generate-docs.sh; then
echo
echo "hack/generate-docs.sh FAILED to complete."
exit 1
fi
fi

if [[ -n "${DRY_RUN}" ]]; then
Expand Down

0 comments on commit 0d88afa

Please sign in to comment.