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

Developer Certificate of Origin workflow #45

Merged
merged 4 commits into from
Sep 28, 2021
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
32 changes: 22 additions & 10 deletions MAINTAINERS.md
Original file line number Diff line number Diff line change
@@ -1,16 +1,20 @@
- [Overview](#overview)
- [Current Maintainers](#current-maintainers)
- [Maintainer Responsibilities](#maintainer-responsibilities)
- [Uphold Code of Conduct](#uphold-code-of-conduct)
- [Prioritize Security](#prioritize-security)
- [Review Pull Requests](#review-pull-requests)
- [Triage Open Issues](#triage-open-issues)
- [Be Responsive](#be-responsive)
- [Maintain Overall Health of the Repo](#maintain-overall-health-of-the-repo)
- [Use Semver](#use-semver)
- [Release Frequently](#release-frequently)
- [Promote Other Maintainers](#promote-other-maintainers)
- [Describe the Repo](#describe-the-repo)
* [Uphold Code of Conduct](#uphold-code-of-conduct)
peternied marked this conversation as resolved.
Show resolved Hide resolved
* [Prioritize Security](#prioritize-security)
* [Review Pull Requests](#review-pull-requests)
* [Triage Open Issues](#triage-open-issues)
* [Be Responsive](#be-responsive)
* [Maintain Overall Health of the Repo](#maintain-overall-health-of-the-repo)
* [Add Continious Integration Checks](#add-continious-integration-checks)
+ [Developer Certificate of Origin workflow](#developer-certificate-of-origin-workflow)
* [Use Semver](#use-semver)
* [Release Frequently](#release-frequently)
* [Promote Other Maintainers](#promote-other-maintainers)
* [Describe the Repo](#describe-the-repo)

<small><i><a href='http://ecotrust-canada.github.io/markdown-toc/'>Table of contents generated with markdown-toc</a></i></small>

## Overview

Expand Down Expand Up @@ -57,6 +61,14 @@ Respond to enhancement requests, and forum posts. Allocate time to reviewing and

Keep the `main` branch at production quality at all times. Backport features as needed. Cut release branches and tags to enable future patches.

### Add Continious Integration Checks
peternied marked this conversation as resolved.
Show resolved Hide resolved

Add integration checks that validate pull requests and pushes to ease the burden on Pull Request reviewers.

#### Developer Certificate of Origin Workflow

Validates pull requests commits are all signed with the doc, [doc.yml](./workflow/doc.yml). Example [pull request](https://github.com/opensearch-project/opensearch-ci/pull/16).

### Use Semver

Use and enforce [semantic versioning](https://semver.org/) and do not let breaking changes be made outside of major releases.
Expand Down
18 changes: 18 additions & 0 deletions workflow/dco.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
name: Developer Certificate of Origin Check

on: [pull_request]

jobs:
check:
runs-on: ubuntu-latest

steps:
- name: Get PR Commits
id: 'get-pr-commits'
uses: tim-actions/get-pr-commits@v1.1.0
with:
token: ${{ secrets.GITHUB_TOKEN }}
- name: DCO Check
uses: tim-actions/dco@v1.1.0
with:
commits: ${{ steps.get-pr-commits.outputs.commits }}