Skip to content

Fix two minor grammar issues#389

Closed
HairyFotr wants to merge 1 commit into
jquery:masterfrom
HairyFotr:master
Closed

Fix two minor grammar issues#389
HairyFotr wants to merge 1 commit into
jquery:masterfrom
HairyFotr:master

Conversation

@HairyFotr
Copy link
Copy Markdown
Contributor

No description provided.

@scottgonzalez
Copy link
Copy Markdown
Member

Thanks. Can you please sign our CLA?

@HairyFotr
Copy link
Copy Markdown
Contributor Author

Done. (Matic Potočnik)

@kswedberg kswedberg closed this in 1b3bc19 Dec 1, 2013
@kswedberg
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor Author

Done.

@scottgonzalez
Copy link
Copy Markdown
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