Skip to content

Commit

Permalink
Updated MAINTAINERS.md format. (opensearch-project#2376)
Browse files Browse the repository at this point in the history
* Updated MAINTAINERS.md to match recommended opensearch-project format.

Signed-off-by: dblock <[email protected]>
  • Loading branch information
dblock authored Jan 5, 2023
1 parent 1ec875c commit 563df8f
Showing 1 changed file with 18 additions and 11 deletions.
29 changes: 18 additions & 11 deletions MAINTAINERS.md
Original file line number Diff line number Diff line change
@@ -1,14 +1,17 @@
- [OpenSearch Security Maintainers](#opensearch-security-maintainers)
- [Maintainers](#maintainers)
- [Updating Practices](#updating-practices)
- [Overview](#overview)
- [Current Maintainers](#current-maintainers)
- [Practices](#practices)
- [Updating Practices](#updating-practices)
- [Reverting Commits](#reverting-commits)
- [Performing Revert](#performing-revert)
- [Squashing a pull request](#squashing-a-pull-request)
- [Performing Revert](#performing-revert)
- [Squashing a Pull Request](#squashing-a-pull-request)

# OpenSearch Security Maintainers
## Overview

This document contains a list of maintainers in this repo. See [opensearch-project/.github/RESPONSIBILITIES.md](https://github.com/opensearch-project/.github/blob/main/RESPONSIBILITIES.md#maintainer-responsibilities) that explains what the role of maintainer means, what maintainers do in this and other repos, and how they should be doing it. If you're interested in contributing, and becoming a maintainer, see [CONTRIBUTING](CONTRIBUTING.md).

## Current Maintainers

## Maintainers
| Maintainer | GitHub ID | Affiliation |
| ---------------- | ----------------------------------------------------- | ----------- |
| Chang Liu | [cliu123](https://github.com/cliu123) | Amazon |
Expand All @@ -18,22 +21,26 @@
| Craig Perkins | [cwperks](https://github.com/cwperks) | Amazon |
| Ryan Liang | [RyanL1997](https://github.com/RyanL1997) | Amazon |

## Practices

### Updating Practices
To ensure common practices as maintainers, all practices are expected to be documented here or enforced through github actions. There should be no expectations beyond what is documented in the repo [CONTRIBUTING.md](./CONTRIBUTING.md) and OpenSearch-Project [CONTRIBUTING.md](https://github.com/opensearch-project/.github/blob/main/CONTRIBUTING.md). To modify an existing processes or create a new one, make a pull request on this MAINTAINERS.md for review and merge it after all maintainers approve of it.

# Practices
To ensure common practices as maintainers, all practices are expected to be documented here or enforced through github actions. There should be no expectations beyond what is documented in the repo [CONTRIBUTING.md](./CONTRIBUTING.md) and OpenSearch-Project [CONTRIBUTING.md](https://github.com/opensearch-project/.github/blob/main/CONTRIBUTING.md). To modify an existing processes or create a new one, make a pull request on this MAINTAINERS.md for review and merge it after all maintainers approve of it.

### Reverting Commits

## Reverting Commits
There will be changes that destabilize or block contributions. The impact of these changes will be localized on the repository or even the entire OpenSearch project. We should bias towards keeping contributions unblocked by immediately reverting impacting changes, these reverts will be done by a maintainer. After the change has been reverted, an issue will be openned to re-merge the change and callout the elements of the contribution that need extra examination such as additional tests or even pull request workflows.

Exceptional, instead of immediately reverting, if a contributor knows how and will resolve the issue in an hour or less we should fix-forward to reduce overhead.

### Performing Revert

Go to the pull request of the change that was an issue, there is a `Revert` button at the bottom. If there are no conflicts to resolve, this can be done immediately bypassing standard approval.

Reverts can also be done via the command line using `git revert <commit-id>` and creating a new pull request. If done in this way they should have references to the pull request that was reverted.

## Squashing a pull request
### Squashing a Pull Request

When a PR is going to be merged, our repositories are set to automatically squash the commits into a single commit. This process needs human intervention to produce high quality commit messages, with the following steps to be followed as much as possible:

- The commit subject is clean and conveys what is being merged
Expand Down

0 comments on commit 563df8f

Please sign in to comment.