-
Notifications
You must be signed in to change notification settings - Fork 317
Update Syntax and spellings. #1161
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
Conversation
@DarkMatter188 Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
* Debug what went wrong on your local machine. | ||
* Test out a a tweak to your `test.sh` script before sending it out. | ||
* Test out for a tweak to your `test.sh` script before sending it out. | ||
* Experiment against a new image before submitting your pull request. |
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.
I'd simply drop the extra "a".
|
||
All of these use cases are great reasons to essentially run what Jenkins does on your local machine. | ||
All of these use cases that are great reasons to know which essentially runs and what Jenkins on your local machine. | ||
|
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.
"All of these use cases are great reasons to run what Jenkins does on your local machine."
@DarkMatter188 Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
Thanks @DarkMatter188. I've taken your patch, polished it, and merged to to |
Backported to |
No description provided.