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

[Doc] Update CONTRIBUTING.md by adding DCO section #337

Merged

Conversation

nung22
Copy link
Contributor

@nung22 nung22 commented Oct 21, 2023

Description

Updates the CONTRIBUTING.md file with more details by adding a section informing contributors about the Developer Certificate of Origin and how to sign off their commits.

Issues Resolved

N/A

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Nicholas Ung <[email protected]>
@codecov
Copy link

codecov bot commented Oct 21, 2023

Codecov Report

Merging #337 (1842a58) into main (77db981) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #337   +/-   ##
=======================================
  Coverage   87.08%   87.08%           
=======================================
  Files          16       16           
  Lines         209      209           
  Branches       43       43           
=======================================
  Hits          182      182           
  Misses         26       26           
  Partials        1        1           
Flag Coverage Δ
dashboards-search-relevance 87.08% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@sejli sejli merged commit 966aed5 into opensearch-project:main Oct 31, 2023
12 checks passed
github-actions bot added a commit that referenced this pull request Oct 31, 2023
Signed-off-by: Nicholas Ung <[email protected]>
(cherry picked from commit 966aed5)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@nung22 nung22 deleted the docs/contributing/add-dco-section branch October 31, 2023 23:14
@noCharger
Copy link
Collaborator

noCharger commented Nov 1, 2023

OpenSearch core and dashboard core both have DCO (For example: https://github.com/opensearch-project/OpenSearch-Dashboards/blob/main/CONTRIBUTING.md#developer-certificate-of-origin). Instead of explicitly stating in plugin repositories, I would take them as a reference. cc @macohen

sejli pushed a commit that referenced this pull request Nov 3, 2023
(cherry picked from commit 966aed5)

Signed-off-by: Nicholas Ung <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants