search

openservicebrokerapi / osb-checker

An automatic checker to verify an Open Service Broker API implementation against the specification
https://github.com/openservicebrokerapi/servicebroker/
Apache License 2.0
48 stars 40 forks source link

Remove trailing whitespaces #53

Closed zhongyi-zhang closed 5 years ago

zhongyi-zhang commented 5 years ago

Resubmit a PR to cover #2.

cfdreddbot commented 5 years ago

Hey zhongyi-zhang!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

mattmcneeney commented 5 years ago

Pullapprove error is gone 😄

norshtein commented 5 years ago

Hi @mattmcneeney , seems @zhongyi-zhang is absent in the approver list...

mattmcneeney commented 5 years ago

@norshtein Fixing that now..

cfdreddbot commented 5 years ago

Hey zhongyi-zhang!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

mattmcneeney commented 5 years ago

I think this is fixed! @zhongyi-zhang may not be showing in the apporvers list for this PR since they created it...

norshtein commented 5 years ago

Thanks @mattmcneeney for the effort!

norshtein commented 5 years ago

LGTM

Approved with PullApprove

zhongyi-zhang commented 5 years ago

Cool. Thanks!

norshtein commented 5 years ago

Reopen the PR to see whether CI effects. Sorry for disturb.

cfdreddbot commented 5 years ago

Hey zhongyi-zhang!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

zhongyi-zhang commented 5 years ago

The number of required approvals is 2. @leonwanghui could you give an approval on this PR?

cfdreddbot commented 5 years ago

Hey zhongyi-zhang!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

mattmcneeney commented 5 years ago

@leonwanghui This needs your +1 again - restarted it to check Travis CI was working (which it is!)

zhongyi-zhang commented 5 years ago

Thank you guys! Merged.