-
-
Notifications
You must be signed in to change notification settings - Fork 5
feat: Update build/release workflows #8
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
Conversation
❌ 10 Tests Failed:
View the top 3 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
678ad31 to
ba71078
Compare
06116fe to
b0e2965
Compare
| @@ -0,0 +1,164 @@ | |||
| --- | |||
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.
FYI some of this file might be able to be axed due to codecov/codecov-cli#664
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.
good shout tyty
| id-token: write # This is required for OIDC | ||
| runs-on: ubuntu-latest | ||
| environment: | ||
| name: pypi |
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.
don't forget to create this environment in GitHub
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.
done ty!
| @@ -1,4 +1,5 @@ | |||
| #!/bin/sh | |||
| cd codecov-cli | |||
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.
only building the codecov-cli? do we need to do this for prevent-cli?
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.
that will be another PR
thomasrockhu-codecov
left a comment
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.
You may want to update the settings on this repo to allow issues also
Updates build and release workflows to work with new package structure and repo.
Notable changes:
I have tested all workflows with the exception of
release_flow.ymlas it needs secrets we don't yet have (working on it).Next PR will add the build steps for prevent-cli, this one is only getting codecov-cli to work.
After this is merged, we should be able to start publishing codecov-cli from this repo exclusively.