Skip to content

Fix two minor grammar issues #389

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

Closed
wants to merge 1 commit into from
Closed

Fix two minor grammar issues #389

wants to merge 1 commit into from

Conversation

HairyFotr
Copy link
Contributor

No description provided.

@scottgonzalez
Copy link
Member

Thanks. Can you please sign our CLA?

@HairyFotr
Copy link
Contributor Author

Done. (Matic Potočnik)

@kswedberg kswedberg closed this in 1b3bc19 Dec 1, 2013
@kswedberg
Copy link
Member

Oof. @scottgonzalez: I think I merged this commit too soon again. Looks like @HairyFotr's git config name/email doesn't match the one in the CLA. They're supposed to use the person's real name and match, right?

@HairyFotr
Copy link
Contributor Author

I missed that in the guidelines. I'm not sure how to include just the updated author info in this pull-request, but I've made a new branch with the correct info here: https://github.com/HairyFotr/api.jquery.com/tree/fix-grammar

Hope I'm not causing too much trouble over such a tiny change...

@mgol
Copy link
Member

mgol commented Dec 1, 2013

@HairyFotr The PR has already been merged so a new one won't help. Please, re-sign the CLA with data you used for this PR. Thanks!

@HairyFotr
Copy link
Contributor Author

Done.

@scottgonzalez
Copy link
Member

@mzgol In the future, please don't ask for duplicate CLA signatures, especially not ones that result in false names. Incorrect commits need to be fixed with .mailmap, see 910e614.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants