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

[PROPOSAL] Delete the manual DCO check #61

Closed
rursprung opened this issue Sep 11, 2023 · 1 comment · Fixed by #92
Closed

[PROPOSAL] Delete the manual DCO check #61

rursprung opened this issue Sep 11, 2023 · 1 comment · Fixed by #92
Labels
enhancement New feature or request

Comments

@rursprung
Copy link
Contributor

What/Why

What are you proposing?

currently, this repository is using the GH action workflow dco.yml to verify that a sign-off is provided. nowadays, this can be controlled directly via GitHub settings (see the GH documentation).
for OpenSearch this has already been changed a year ago by @VachaShah in opensearch-project/OpenSearch#2317

if this is done it should also be documented that the same needs to be done on repositories started off of this template (presuming they want the DCO).

What users have asked for this feature?

n/a

What problems are you trying to solve?

align repositories, use standard features of GitHub.

What is the developer experience going to be?

n/a

Are there any security considerations?

n/a

Are there any breaking changes to the API

n/a

What is the user experience going to be?

n/a

Are there breaking changes to the User Experience?

n/a

Why should it be built? Any reason not to?

n/a

What will it take to execute?

  • admin user to set the setting on the GH repo
  • PR to remove the DCO workflow

Any remaining open questions?

n/a

@dblock
Copy link
Member

dblock commented Sep 12, 2023

We have a DCO bot enabled for the org. The manual DCO check can be deleted.

@dblock dblock added enhancement New feature or request and removed untriaged labels Jun 6, 2024
@dblock dblock changed the title [PROPOSAL] move DCO check to GitHub setting [PROPOSAL] Delete the manual DCO check Jun 6, 2024
rursprung added a commit to rursprung/opensearch-plugin-template-java that referenced this issue Jun 20, 2024
the DCO check is activated directly on GitHub in the org-level settings
for `opensearch-project` (see the [GitHub documentation]). there's no
need for the duplicate check implemented in the workflow anymore.

other projects which are created based on the template which want a DCO
check can also just enable this in their settings.

fixes opensearch-project#61

[GitHub documentation]: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/managing-repository-settings/managing-the-commit-signoff-policy-for-your-repository
@reta reta closed this as completed in #92 Jun 20, 2024
@reta reta closed this as completed in 915063d Jun 20, 2024
rursprung added a commit to rursprung/opensearch-plugin-template-java that referenced this issue Jun 20, 2024
the DCO check is activated directly on GitHub in the org-level settings
for `opensearch-project` (see the [GitHub documentation]). there's no
need for the duplicate check implemented in the workflow anymore.

other projects which are created based on the template which want a DCO
check can also just enable this in their settings.

fixes opensearch-project#61

[GitHub documentation]: https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/managing-repository-settings/managing-the-commit-signoff-policy-for-your-repository

Signed-off-by: Ralph Ursprung <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants