Closed dprotaso closed 4 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dprotaso
The full list of commands accepted by this bot can be found here.
The pull request process is described here
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 79.12%. Comparing base (
f8ed9a7
) to head (8cd0a8b
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
/test integration-tests-contour_net-gateway-api_main
/assign @izabelacg
This is ready - but the diff is too large so a bunch of checks fail - we can ignore those. For me the main thing is the integration tests pass.
/lgtm
Overriding broken jobs - diff is to large and reviewdog breaks
/override "style / Golang / Lint" /override "style / Golang / Do Not Submit" /override "style / Golang / Boilerplate Check (go)" /override "style / Golang / Boilerplate Check (sh)" /override "style / suggester / shell" /override "codecov/patch"
@dprotaso: /override requires failed status contexts, check run or a prowjob name to operate on. The following unknown contexts/checkruns were given:
codecov/patch
Only the following failed contexts/checkruns were expected:
EasyCLA
Generate some code!
build-tests_net-gateway-api_main
integration-tests-contour_net-gateway-api_main
integration-tests-istio_net-gateway-api_main
style / Golang / Boilerplate Check (go)
style / Golang / Boilerplate Check (sh)
style / Golang / Do Not Submit
style / Golang / Lint
style / suggester / github_actions
style / suggester / shell
style / suggester / yaml
tide
unit-tests_net-gateway-api_main
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.
/override "patch"
@dprotaso: /override requires failed status contexts, check run or a prowjob name to operate on. The following unknown contexts/checkruns were given:
patch
Only the following failed contexts/checkruns were expected:
EasyCLA
Generate some code!
build-tests_net-gateway-api_main
integration-tests-contour_net-gateway-api_main
integration-tests-istio_net-gateway-api_main
style / Golang / Boilerplate Check (go)
style / Golang / Boilerplate Check (sh)
style / Golang / Do Not Submit
style / Golang / Lint
style / suggester / github_actions
style / suggester / shell
style / suggester / yaml
tide
unit-tests_net-gateway-api_main
If you are trying to override a checkrun that has a space in it, you must put a double quote on the context.
/override "style / Golang / Lint" /override "style / Golang / Do Not Submit" /override "style / Golang / Boilerplate Check (go)" /override "style / Golang / Boilerplate Check (sh)" /override "style / suggester / shell"
@dprotaso: Overrode contexts on behalf of dprotaso: style / Golang / Boilerplate Check (go), style / Golang / Boilerplate Check (sh), style / Golang / Do Not Submit, style / Golang / Lint, style / suggester / shell
Trying to redo - https://github.com/knative-extensions/net-gateway-api/pull/729