ci: add ci to test the commands #207
Conversation
|
Skipping CI for Draft Pull Request. |
|
/test all |
subhamkrai
force-pushed
the
add-ci
branch
2 times, most recently
from
3a8f136 to
284a9b5
Compare
|
/test all |
|
@subhamkrai: No presubmit jobs available for red-hat-storage/odf-cli@main DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
subhamkrai
changed the title
|
/test all |
subhamkrai
changed the title
|
/lgtm |
|
@malayparida2000: changing LGTM is restricted to collaborators DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
subhamkrai
force-pushed
the
add-ci
branch
2 times, most recently
from
64f31bb to
e7e3b0b
Compare
subhamkrai
force-pushed
the
add-ci
branch
8 times, most recently
from
a0cb44e to
c9c737e
Compare
subhamkrai
requested review from
nirs and
travisn
and removed request for
nirs
subhamkrai
force-pushed
the
add-ci
branch
3 times, most recently
from
0535bf2 to
d68c1cd
Compare
|
@travisn ^ |
travisn
left a comment
travisn
left a comment
There was a problem hiding this comment.
A couple of the new CI tests are failing?
subhamkrai
force-pushed
the
add-ci
branch
3 times, most recently
from
8c2ba69 to
73d01f1
Compare
This commit add ci tests for the go commands, adds golangci test and commitlint ci. Signed-off-by: subhamkrai <[email protected]>
| @@ -0,0 +1,29 @@ | |||
| name: Commitlint | |||
There was a problem hiding this comment.
I'd suggest let's not have the commitlint in this repo, I don't think we need it for the downstream.
| # actions/setup-go already handles caching | ||
| skip-cache: true | ||
|
|
||
| govulncheck: |
There was a problem hiding this comment.
If this is not passing yet, let's remove it from this PR and have a follow-up PR. It's better to keep green CI when introducing new checks.
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: odf-cli test |
There was a problem hiding this comment.
This test is failing. Either let's find the fix, or save for a follow-up PR.
6 participants
ci: add ci to test the commands