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
Ginkgo: Simplify all --focus regexes #12089
Conversation
test-me-please |
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.
One nit below. LGTM otherwise!
Thanks for fixing this Jarno! 🎉 It was really confusing when I started.
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.
Some small nits, otherwise LGTM.
Looks good, I would apply changes suggested by @tklauser though. |
Ginkgo --focus option is a regular expression that matches a substring without any trailing wildcarding. Simplify all --focus parameters accordingly. The --focus parameter is also not repeatable, the last one takes the effect if multiple ones are given. Fix the docs for this. Update the document to the current default K8S_VERSION (1.18). Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
464905b
to
e295283
Compare
test-me-please |
Ginkgo --focus option is a regular expression that matches a substring
without any trailing wildcarding. Simplify all --focus parameters
accordingly. The --focus parameter is also not repeatable, the last
one takes the effect if multiple ones are given. Fix the docs for
this.
Update the document to the current default K8S_VERSION (1.18).
Signed-off-by: Jarno Rajahalme jarno@covalent.io