Skip to content
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

Fix and improve Conformance Ginkgo UX #25950

Merged
merged 2 commits into from Jun 7, 2023
Merged

Conversation

aanm
Copy link
Member

@aanm aanm commented Jun 6, 2023

In order for needs..outputs. work, the job needs to depend on the job that is being checked.

Fixes: 79e1c31 (".github: add missing job to check for code changes")

@aanm aanm added area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. release-note/misc This PR makes changes that have no direct user impact. labels Jun 6, 2023
@aanm aanm requested review from a team as code owners June 6, 2023 19:04
@aanm aanm requested a review from brlbil June 6, 2023 19:04
aanm added 2 commits June 6, 2023 22:53
In order for needs.*.outputs.* work, the job needs to depend on the job
that is being checked.

Fixes: 79e1c31 (".github: add missing job to check for code changes")
Signed-off-by: André Martins <andre@cilium.io>
To be consistent with other workflows that use matrixes,
conformance-e2e.yaml, we should also report the workflow status in a
similar way.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm changed the title .github: fix conditional check on conformance ginkgo Fix and improve Conformance Ginkgo UX Jun 6, 2023
Copy link
Contributor

@brlbil brlbil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aanm aanm merged commit e4cf16a into cilium:main Jun 7, 2023
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/misc Impacts miscellaneous areas of the code not otherwise owned by another area. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants