-
Notifications
You must be signed in to change notification settings - Fork 33
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
Bump to ginkgo2 #58
Bump to ginkgo2 #58
Conversation
-remove color feature cause couldn't understand how to use it in v2 Authored-by: Michael Oleske <[email protected]>
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.
We probably don't have to get rid of our ReporterConfig interactions.
Otherwise LGTM. Thanks @moleske!
- this is because you need to pass new reporter config to run specs - I don't like this because the seperation seems arbitrary without knowing how ginkgo v2 wants things setup - kept the tests neseted in command reporter to at least kinda keep the suites together Authored-by: Michael Oleske <[email protected]>
IVe created my own fork to move to https://github.com/KevinJCross/cf-test-helpers we are actively using this fork in https://github.com/cloudfoundry/app-autoscaler-release seems stable and good enough to us. |
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.
LGTM thanks @moleske !
-remove color feature cause couldn't understand how to use it in v2
Authored-by: Michael Oleske [email protected]