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

Migrating to DCO 1.1 #47

Closed
wants to merge 2 commits into from
Closed

Migrating to DCO 1.1 #47

wants to merge 2 commits into from

Conversation

mikeal
Copy link
Contributor

@mikeal mikeal commented Feb 9, 2016

We should use the latest version of important legal documents.

@@ -129,21 +129,31 @@ objections to adding or removing individual members, an effort must be
made to resolve those objections. If consensus cannot be reached, a
vote may be called following the process above.

### Developer's Certificate of Origin 1.0
# Developer's Certificate of Origin 1.1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needs to be ###

@jasnell
Copy link
Member

jasnell commented Feb 9, 2016

LGTM

@rvagg
Copy link
Member

rvagg commented Feb 9, 2016

lgtm sans formatting nit

makes me wonder if there was some drama that caused the introduction of clause (d), some incident perhaps that caused a flare up somewhere because a company didn't realise contributions would be public and stick around forever

@thefourtheye
Copy link
Contributor

LGTM

@williamkapke
Copy link
Contributor

Whoops. My PR #95 did the same as this one. Sry to steal your thunder @mikeal ;)

@sam-github sam-github deleted the dco-1.1 branch December 5, 2019 16:17
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.

5 participants