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

[DO NOT MERGE] Test DCO #70

Closed
wants to merge 2 commits into from
Closed

[DO NOT MERGE] Test DCO #70

wants to merge 2 commits into from

Conversation

agalbachicar
Copy link
Member

🎉 New feature

Closes #51

Summary

Test to validate DCO configuration.

Test it

Checks over the commits.

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@agalbachicar
Copy link
Member Author

DCO failed on a314012 because it was set to validate all commits.

Signed-off-by: Agustin Alba Chicar <ag.albachicar@gmail.com>
Signed-off-by: Agustin Alba Chicar <ag.albachicar@gmail.com>
@agalbachicar agalbachicar force-pushed the agalbachicar/test_dco branch from a314012 to 096d84f Compare October 18, 2024 08:05
@agalbachicar
Copy link
Member Author

After fixing the unsigned commit DCO was fixed.

@agalbachicar agalbachicar deleted the agalbachicar/test_dco branch October 18, 2024 08:05
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.

Enable DCO
1 participant