-
Notifications
You must be signed in to change notification settings - Fork 39
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
chore: consistent naming of upload options #473
Conversation
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
Makes the `--report-code` of `do-upload` to also accept `--code`. This makes it consistent with other commands. closes codecov/feedback#380
bda9e20
to
a513878
Compare
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
2 similar comments
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
Test Failures Detected: Due to failing tests, we cannot provide coverage reports at this time. ❌ Failed Test Results:Completed 3465 tests with View the full list of failed testspytest
|
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
1 similar comment
❌ Failed Test Results:Completed 692 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
4 similar comments
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
4 similar comments
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
de2470b
to
3a859ac
Compare
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
4 similar comments
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
❌ Failed Test Results:Completed 693 tests with View the full list of failed tests
|
@@ -31,8 +31,7 @@ jobs: | |||
- os: ubuntu-20.04 | |||
TARGET: ubuntu | |||
CMD_REQS: > | |||
pip install -r requirements.txt | |||
pip install . | |||
pip install -r requirements.txt && pip install . |
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.
@giovanni-guidini FYI, both deps could be passed to the same pip install
command. There's no technical reason for running it twice here.
Makes the
--report-code
ofdo-upload
to also accept--code
. This makes it consistent with other commands.closes codecov/feedback#380