Skip to content
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

Add CLA link #3020

Merged
merged 2 commits into from Sep 27, 2017
Merged

Add CLA link #3020

merged 2 commits into from Sep 27, 2017

Conversation

skeggse
Copy link
Contributor

@skeggse skeggse commented Sep 27, 2017

New contributors may wish to review the CLA prior to working on a pull request. Currently, the only obvious way to view the CLA is to open a pull request.

@jsf-clabot
Copy link

jsf-clabot commented Sep 27, 2017

CLA assistant check
All committers have signed the CLA.

@boneskull
Copy link
Member

in the future I'd recommend adding [ci skip] to the commit message to avoid builds on a documentation change

@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 89.072% when pulling 0855783 on skeggse:patch-2 into e39a867 on mochajs:master.

@boneskull boneskull added this to the v4.0.0 milestone Sep 27, 2017
@boneskull
Copy link
Member

thanks again

@boneskull boneskull merged commit 86e76f9 into mochajs:master Sep 27, 2017
@ScottFreeCode
Copy link
Contributor

in the future I'd recommend adding [ci skip] to the commit message to avoid builds on a documentation change

Is this mentioned in our CONTRIBUTING and PR TEMPLATE guides? If not we should add it too!

sgilroy pushed a commit to TwineHealth/mocha that referenced this pull request Feb 27, 2019
* Add link to CLA in README

* Add CLA link to CONTRIBUTING document
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants