-
Notifications
You must be signed in to change notification settings - Fork 589
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
Best practices for testing #997
Conversation
@TessFerrandez, the MegaLinter problems all seem to be related to articles other than the stuff I am submitting for. Any thoughts? |
Hi @plasne |
I'm fixing the broken links in #998 so please approve that and then merge it into yours |
Thank you for the contribution! |
…icrosoft/code-with-engineering-playbook into pelasne/build-for-testing
…/code-with-engineering-playbook into pelasne/build-for-testing
Thanks, @TessFerrandez , merging that branch seemed to work. |
@stuartleeks can you review again, I think all the comments are addressed |
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!
This PR adds the following:
This comes from the hydro crew's latest engagement and we believe might be helpful to other crews.