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
docs: add documentation for Ginkgo-based GHA #26055
Conversation
b213df8
to
30f046a
Compare
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.
The PRs can be merged independently.
I'd rather have these YAML files merged in before referencing them in the docs, for my part.
Nice work! I've got some comments below. Mostly minor items, however one concern is that we could probably integrate the new section for e2e tests better. I'd probably feel lost landing on the new version of the page if I were trying to get started with e2e tests, see more comments below.
Direct links to preview:
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.
Thanks for the review Quentin!
30f046a
to
7670fb9
Compare
c873253
to
fb1eae9
Compare
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.
Looks like all items were addressed, thanks!
One more comment on the numbering for the list (see above), but looks all good otherwise, thanks a lot!
This commit adds comprehensive documentation on how to use GitHub Actions for Ginkgo Workflows, how to run tests locally, and test a particular SHA using GitHub. Signed-off-by: André Martins <andre@cilium.io>
fb1eae9
to
b112828
Compare
I note that #26019 has been merged, so we're good to go about referencing the new YAML files. |
This commit adds comprehensive documentation on how to use GitHub Actions for Ginkgo Workflows, how to run tests locally, and test a particular SHA using GitHub.
The docs were written on top of #26019. The PRs can be merged independently.