-
Notifications
You must be signed in to change notification settings - Fork 11.2k
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
Run simtest code coverage without --config flags #16134
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
4 Ignored Deployments
|
# SPDX-License-Identifier: Apache-2.0 | ||
|
||
# verify that git repo is clean | ||
if [[ -n $(git status -s) ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mystenmark, why do we need this? This workflow is going to be run on ubuntu-ghcloud
GH runner. So there should never be the case when repo is not clean
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
someone could run the script locally (e.g. to test it) and it could clobber local files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
./scripts/simtest/codecov.sh
shell: /usr/bin/bash -e {0}
env:
CARGO_TERM_COLOR: always
CARGO_INCREMENTAL: 0
CARGO_NET_RETRY: 10
RUSTUP_MAX_RETRIES: 10
RUST_BACKTRACE: short
RUSTDOCFLAGS: -D warnings
CACHE_ON_FAILURE: false
Working directory is not clean. Please commit all changes before running this script. ./scripts/simtest/codecov.sh
shell: /usr/bin/bash -e {0}
env:
CARGO_TERM_COLOR: always
CARGO_INCREMENTAL: 0
CARGO_NET_RETRY: 10
RUSTUP_MAX_RETRIES: 10
RUST_BACKTRACE: short
RUSTDOCFLAGS: -D warnings
CACHE_ON_FAILURE: false
Working directory is not clean. Please commit all changes before running this script.
Is what I was afraid of when this script gets run. I feel like these are two different cases here
No description provided.