openservicebrokerapi / servicebroker

Open Service Broker API Specification
https://openservicebrokerapi.org/
Apache License 2.0
1.19k stars 434 forks source link

Fixing minor typos. #324

Closed n3wscott closed 7 years ago

n3wscott commented 7 years ago

Reading the spec for the first time, these jumped out as minor mistakes.

cfdreddbot commented 7 years ago

Hey n3wscott!

Thanks for submitting this pull request!

All pull request submitters and commit authors must have a Contributor License Agreement (CLA) on-file with us. Please sign the appropriate CLA (individual or corporate).

When sending signed CLA please provide your github username in case of individual CLA or the list of github usernames that can make pull requests on behalf of your organization.

If you are confident that you're covered under a Corporate CLA, please make sure you've publicized your membership in the appropriate Github Org, per these instructions.

Once you've publicized your membership, one of the owners of this repository can close and reopen this pull request, and dreddbot will take another look.

duglin commented 7 years ago

once the CLA is signed, I'm ok with these edits. LGTM

Approved with PullApprove

cfdreddbot commented 7 years ago

Hey n3wscott!

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.

duglin commented 7 years ago

hmm @mattmcneeney shouldn't those links be good now?

mattmcneeney commented 7 years ago

@duglin yes they should. I think the problem may be that this branch needs to be rebased on master?

n3wscott commented 7 years ago

Ok let me try to rebase.

n3wscott commented 7 years ago

I think I had rebased already but I had not pushed my local master to github. Done now.

mattmcneeney commented 7 years ago

Looks like it's going to pass this time :)

vaikas commented 7 years ago

LGTM

Approved with PullApprove

duglin commented 7 years ago

LGTM

Approved with PullApprove

mattmcneeney commented 7 years ago

LGTM

Approved with PullApprove

angarg12 commented 7 years ago

LGTM

Approved with PullApprove