Skip to content

add DCO.txt and remove mergify DCO requirement #275

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add DCO.txt and remove mergify DCO requirement #275

wants to merge 2 commits into from

Conversation

ktdreyer
Copy link
Contributor

@ktdreyer ktdreyer commented Jun 2, 2025

Add the contents of https://developercertificate.org/ to DCO.txt at the root of this repository.

Explain to contributors that new submissions imply acceptance of the DCO.

Related: instructlab/dev-docs#192

Add the contents of https://developercertificate.org/ to DCO.txt at the
root of this repository.

Explain to contributors that new submissions imply acceptance of the
DCO.

Signed-off-by: Ken Dreyer <[email protected]>
@mergify mergify bot added the documentation Improvements or additions to documentation label Jun 2, 2025
@booxter
Copy link
Contributor

booxter commented Jun 2, 2025

Please update mergify rules to not require DCO.

@ktdreyer
Copy link
Contributor Author

ktdreyer commented Jun 3, 2025

I plan to do that after we merge this.

@mergify mergify bot added the one-approval label Jun 4, 2025
We are going to disable this job for this repository and rely on
contributors' implicit acceptance of DCO.txt instead.

Signed-off-by: Ken Dreyer <[email protected]>
@ktdreyer
Copy link
Contributor Author

ktdreyer commented Jun 4, 2025

You were right @booxter . It makes sense to disable the mergify rule here.

@mergify mergify bot added the CI/CD Affects CI/CD configuration label Jun 4, 2025
@ktdreyer ktdreyer changed the title add DCO.txt add DCO.txt and remove mergify DCO requirement Jun 4, 2025
@mergify mergify bot removed the one-approval label Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Affects CI/CD configuration documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants