Skip to content

Use strict error handling in bash scripts #3083

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.

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

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

NickVolynkin
Copy link
Contributor

This will help detect failures in docs CI/CD
as soon as possible.

@github-actions github-actions bot temporarily deployed to branch-bash-xe-pipefail-nounset August 10, 2022 12:01 Inactive
@NickVolynkin NickVolynkin force-pushed the nickvolynkin/bash-xe-pipefail-nounset branch from b8482ac to e56e293 Compare August 10, 2022 12:24
@github-actions github-actions bot temporarily deployed to branch-bash-xe-pipefail-nounset August 10, 2022 12:28 Inactive
@NickVolynkin NickVolynkin force-pushed the nickvolynkin/bash-xe-pipefail-nounset branch from e56e293 to 8108929 Compare August 10, 2022 12:35
@github-actions github-actions bot temporarily deployed to branch-bash-xe-pipefail-nounset August 10, 2022 12:37 Inactive
@NickVolynkin NickVolynkin force-pushed the nickvolynkin/bash-xe-pipefail-nounset branch from 8108929 to c42a7fd Compare August 10, 2022 13:04
@github-actions github-actions bot temporarily deployed to branch-bash-xe-pipefail-nounset August 10, 2022 13:06 Inactive
@NickVolynkin NickVolynkin force-pushed the nickvolynkin/bash-xe-pipefail-nounset branch from c42a7fd to 3e79aca Compare August 10, 2022 13:12
@github-actions github-actions bot temporarily deployed to branch-bash-xe-pipefail-nounset August 10, 2022 13:14 Inactive
@NickVolynkin NickVolynkin force-pushed the nickvolynkin/bash-xe-pipefail-nounset branch from 3e79aca to ddd37d2 Compare August 10, 2022 13:22
@github-actions github-actions bot temporarily deployed to branch-bash-xe-pipefail-nounset August 10, 2022 13:24 Inactive
@NickVolynkin NickVolynkin force-pushed the nickvolynkin/bash-xe-pipefail-nounset branch from ddd37d2 to 5ad5ce4 Compare August 10, 2022 13:31
@github-actions github-actions bot temporarily deployed to branch-bash-xe-pipefail-nounset August 10, 2022 13:33 Inactive
@NickVolynkin NickVolynkin requested a review from foxzi August 10, 2022 13:45
This will help detect failures in docs CI/CD
 as soon as possible.

* Upload PDF and singlehtml only if these formats were
  built before.
* Remove the `yes | command` construction which fails with
  `set -xe -o pipefail`.
* Add `-v` to file copying for more informative logs.
@NickVolynkin NickVolynkin force-pushed the nickvolynkin/bash-xe-pipefail-nounset branch from 5ad5ce4 to c9b2494 Compare August 10, 2022 13:47
@github-actions github-actions bot temporarily deployed to branch-bash-xe-pipefail-nounset August 10, 2022 13:50 Inactive
@NickVolynkin NickVolynkin merged commit 1167609 into latest Aug 10, 2022
@NickVolynkin NickVolynkin deleted the nickvolynkin/bash-xe-pipefail-nounset branch August 10, 2022 14:51
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