Skip to content

Conversation

@GHkrishna
Copy link
Contributor

@GHkrishna GHkrishna commented Jul 29, 2025

Ability to add remediation commits as per: https://github.com/cncf/dco2?tab=readme-ov-file#remediation-commits

Adding this to main should ideally solve the issue, as per this discussion: cncf/dco2#89


Important

Adds .github/dco.yml to enable remediation commits for individual and third-party contributions.

  • Configuration:
    • Adds .github/dco.yml to enable remediation commits.
    • Sets allowRemediationCommits for individual and thirdParty to true.

This description was created by Ellipsis for 4ee3586. You can customize this summary. It will automatically update as commits are pushed.

@GHkrishna GHkrishna self-assigned this Jul 29, 2025
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

Important

Looks good to me! 👍

Reviewed everything up to 4ee3586 in 25 seconds. Click for details.
  • Reviewed 9 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. .github/dco.yml:1
  • Draft comment:
    The configuration for remediation commits looks correct; consider adding inline comments or README updates to clarify the purpose of each setting for future maintainers.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50% None

Workflow ID: wflow_si1XcT6IPV5cmX3v

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@sonarqubecloud
Copy link

@amitpadmani-awts amitpadmani-awts merged commit 915f45c into main Jul 29, 2025
4 of 5 checks passed
@amitpadmani-awts amitpadmani-awts deleted the chore/provision-to-add-remediation-commit branch July 29, 2025 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants