Conversation
|
@rhcarvalho thanks for fixing this. Could you please squash into one commit, since basically both commits are just fixing typos ? |
rhcarvalho
force-pushed
the
typokiller
branch
from
a17b91f to
5f49d6e
Compare
|
@jhadvig I had it separate because the capitalization cases seemed debatable. I've squashed it all into one commit now. |
|
@rhcarvalho thanks. LGTM |
|
[test] |
|
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_requests_openshift3/1059/) |
|
[merge] |
1 similar comment
|
[merge] |
|
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_requests_openshift3/921/) (Image: devenv-fedora_800) |
|
Evaluated for origin up to 5f49d6e |
Merged by openshift-bot
|
@rhcarvalho thanks for doing this! |
|
@rhcarvalho can you do the same for https://github.com/openshift/source-to-image ? ;-) |
|
@mfojtik 😋 yes |
* Add new Checksum field to BrokerStatus This adds a check to run the broker reconciler if the spec has changed via monitoring of the check sum field. Closes openshift#1018 * Generated files
4 participants
Hi everyone! I used OpenShift Origin's repo as my first test target for typokiller.
Happy to see just a small amount of typos, kudos to you all! Great job!