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

CONTRIBUTING: Replace CLA with GitHub ToS notice #218

Merged
merged 3 commits into from
Sep 24, 2017
Merged

CONTRIBUTING: Replace CLA with GitHub ToS notice #218

merged 3 commits into from
Sep 24, 2017

Conversation

mbland
Copy link
Owner

@mbland mbland commented Sep 24, 2017

Closes #217.

Also cleans up the formatting of CONTRIBUTING.md a bit, and updates its advice on process substitutions.

Closes #217.

Also cleans up the formatting of CONTRIBUTING.md a bit, and updates its
advice on process substitutions.
@mbland mbland added this to the v1.7.0 milestone Sep 24, 2017
@mbland mbland self-assigned this Sep 24, 2017
Since the template gets added after the text from a single-commit pull
request, it seems to make more sense to just remove the issue reference
and description bits, as those should be included in the commit message
to begin with.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.145% when pulling d684fe0 on cla-#217 into dde3baf on master.

Forgot to include `blob/master` in the URLs.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.145% when pulling bb2f76e on cla-#217 into dde3baf on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.145% when pulling bb2f76e on cla-#217 into dde3baf on master.

@mbland mbland merged commit d4d6fc2 into master Sep 24, 2017
@mbland mbland deleted the cla-#217 branch September 24, 2017 17:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants