Skip to content

Explicit HTTPS calls#116

Closed
alanyee wants to merge 1 commit intojquery:masterfrom
alanyee:patch-1
Closed

Explicit HTTPS calls#116
alanyee wants to merge 1 commit intojquery:masterfrom
alanyee:patch-1

Conversation

@alanyee
Copy link
Contributor

@alanyee alanyee commented Dec 19, 2015

Following recent guidelines to make explicit HTTPS calls

Following recent guidelines to make explicit HTTPS calls
@jquerybot
Copy link

Thank you for your pull request. It looks like this may be your first contribution to a jQuery Foundation project, if so we need you to sign our Contributor License Agreement (CLA).

📝 Please visit http://contribute.jquery.org/CLA/ to sign.

After you signed, the PR is checked again automatically after a minute. If there's still an issue, please reply here to let us know.


If you've already signed our CLA, it's possible your git author information doesn't match your CLA signature (both your name and email have to match), for more information, check the status of your CLA check.

@alanyee
Copy link
Contributor Author

alanyee commented Dec 19, 2015

I just signed the CLA.

@AurelioDeRosa
Copy link
Member

Hi @alanyee. Can you please rebase your PR so that we can merge it?

@alanyee
Copy link
Contributor Author

alanyee commented Mar 15, 2016

In what way do you want me to rebase it?

@AurelioDeRosa
Copy link
Member

I meant using git rebase.

@mgol
Copy link
Member

mgol commented Mar 23, 2016

@alanyee Do you need some further help?

@alanyee
Copy link
Contributor Author

alanyee commented Mar 23, 2016

@mgol Perhaps, I am not how to use git rebase.

@mgol mgol closed this in e37ea3f Mar 30, 2016
@mgol
Copy link
Member

mgol commented Mar 30, 2016

@alanyee I rebased and landed it for you. For the sake of the future you may read our "Commits and Pull Requests" guide and the "Rewriting History" Atlassian tutorial to learn about the subject.

Thanks!

@alanyee alanyee deleted the patch-1 branch August 28, 2017 16:53
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