]> git.lizzy.rs Git - rust.git/commit - src/tools/miri
Rollup merge of #74418 - rye:gha-dedup-shell-setting, r=pietroalbini
authorManish Goregaokar <manishsmail@gmail.com>
Sat, 18 Jul 2020 01:13:42 +0000 (18:13 -0700)
committerGitHub <noreply@github.com>
Sat, 18 Jul 2020 01:13:42 +0000 (18:13 -0700)
commit7b66c6678c107b4b391a8561c375e8ffad0f52e2
tree3d54a63d3c367860e071ee826ca1e96b5e3fb251
parente775b4dbc4a991d60014225088c58eddd91f18cb
parent586629c0b607f34981a893cf856bb35279f58542
Rollup merge of #74418 - rye:gha-dedup-shell-setting, r=pietroalbini

ci: Set `shell: bash` as a default, remove duplicates

A follow-up to #74406, this commit merely removes the `shell: bash` lines where they were added in favor of setting defaults for *all* "run" steps in the jobs that run the tests.

The changes in #74406 were needed because of an upstream change to the `windows-2019` GitHub Actions image. Previously, the configuration worked fine without specifying `shell: bash`, but for some reason this broke with a new change that was deployed today. The preceding PR was a hotfix to get CI passing, but there was a slightly less duplicative way to specify the default shell for the jobs, which was to set the `defaults.run` option.

This change applies to the `pr`, `try`, `auto`, and `auto-fallible` jobs, which are derived from the YAML-anchor `base-ci-job`.  I did not apply these changes to the `master`, `try-success`, `try-failure`, `auto-success`, or `auto-failure` jobs because they have only a few steps.

cc/r? @Mark-Simulacrum