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

turn on DCO #82

Closed
dhh1128 opened this issue Jul 12, 2019 · 8 comments
Closed

turn on DCO #82

dhh1128 opened this issue Jul 12, 2019 · 8 comments

Comments

@dhh1128
Copy link
Collaborator

dhh1128 commented Jul 12, 2019

I would like to turn on the DCO bot, and to go back through history and add a Signed-off-by note to any commits that lack it. Do any of our contributors object? @kdenhartog @TelegramSam @csmarc @Oskar-van-Deventer @SvenHammann90 @devin-fisher @brentzundel @mavarley

Background:

It's not clear whether this spec will end up being sponsored in a formal way by any standards body or open source umbrella project--but it seems moderately likely that one or more of the W3C, DIF, and/or Hyperledger might endorse it officially.

If that happens, one of the issues that will arise is whether all contributors to a spec are officially aligned with this repo's Apache 2 license (which has always been here). Specifically, they will want to be sure that nothing in the spec is encumbered by copyright or patents in any way.

Github has a way to officially prove that, which is through the DCO bot. I can turn that on now, which will begin enforcement from this point forward. I can also modify the log message from any old commits to clarify that this was the case for them, too. But I don't want to modify old commit messages from any of you unless you are comfortable with me doing so.

For reference, this is common practice in Hyperledger, where I have done a lot of work, because it makes all the lawyers happy and it forces collaborating developers to be crisp about IP.

@SvenHammann90
Copy link
Contributor

I agree to this, including modification of my old commit messages.

Kind Regards,
Sven

@kdenhartog
Copy link
Contributor

I believe I've been using DCO commit methods anyways. I'll double check my commits and amend them if they have problems.

@brentzundel
Copy link
Contributor

brentzundel commented Jul 15, 2019 via email

@csmarc
Copy link
Contributor

csmarc commented Jul 16, 2019

I agree to this as well, and I turned on DCO.

@Oskar-van-Deventer
Copy link
Contributor

I have no objection at turning on DCO, though I may need some help to get my PR's properly DCO-ed ...

By the way, what refrains us from moving the whole Peer DID spec to Hyperledger Aries?
(I don't know the history of openssi, so please pardon my ignorance)

Oskar

@dhh1128
Copy link
Collaborator Author

dhh1128 commented Jul 16, 2019

Moving this spec to a more permanent home is probably a good idea. However, all the candidates for a permanent homes had political problems when I began the work. Therefore, I deliberately placed this spec in the most generic, most unaffiliated place I could think of, for the time being, because I wanted it not to be strongly associated with any particular sponsor. I wanted people who encountered it to just evaluate it on the basis of its ideas, and to defer the question of its home (which is secondary) until later. Now that we have a significant collection of authors, I think we can ask them where they want to move it, and start developing a consensus about that question. I'll open a separate ticket about that.

@dhh1128
Copy link
Collaborator Author

dhh1128 commented Jul 15, 2020

@peacekeeper , I can't see the settings on our new repo at DIF. Can you confirm that DCO check is enabled there?

@dhh1128
Copy link
Collaborator Author

dhh1128 commented Jul 15, 2020

Opened a related issue in the new repo home: decentralized-identity/peer-did-method-spec#18.

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

No branches or pull requests

6 participants