-
Notifications
You must be signed in to change notification settings - Fork 360
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
docs: Updates issue and pull request templates (#189)
Signed-off-by: aeneasr <[email protected]>
- Loading branch information
Showing
6 changed files
with
74 additions
and
87 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,5 +4,5 @@ about: Please do not open a public issue about the vulnerability but instead dis | |
|
||
--- | ||
|
||
Please refrain from publishing (potential) security vulnerabilities publicly on the forums, the chat, or GitHub. Instead, | ||
send us an email to [[email protected]](mailto:[email protected]) and we will respond within 24 hours. | ||
**Please refrain from publishing (potential) security vulnerabilities publicly on the forums, the chat, or GitHub. Instead, | ||
send us an email to [[email protected]](mailto:[email protected]) and we will respond within 24 hours.** |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -27,10 +27,6 @@ them, don't hesitate to ask. We're here to help! This is simply a reminder of wh | |
- [ ] I have read the [contributing guidelines](CONTRIBUTING.md) | ||
- [ ] I confirm that this pull request does not address a security vulnerability. If this pull request addresses a security | ||
vulnerability, I confirm that I got green light (please contact [[email protected]](mailto:[email protected])) from the maintainers to push the changes. | ||
- [ ] I signed the [Developer's Certificate of Origin](https://github.com/ory/keto/blob/master/CONTRIBUTING.md#developers-certificate-of-origin) | ||
by signing my commit(s). You can amend your signature to the most recent commit by using `git commit --amend -s`. If you | ||
amend the commit, you might need to force push using `git push --force HEAD:<branch>`. Please be very careful when using | ||
force push. | ||
- [ ] I have added tests that prove my fix is effective or that my feature works | ||
- [ ] I have added necessary documentation within the code base (if appropriate) | ||
- [ ] I have documented my changes in the [developer guide](https://github.com/ory/docs) (if appropriate) | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
# Contribution Guide | ||
# Contributing to ORY Oathkeeper | ||
|
||
<!-- START doctoc generated TOC please keep comment here to allow auto update --> | ||
<!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE --> | ||
|
@@ -8,7 +8,6 @@ | |
- [Contributing Code](#contributing-code) | ||
- [Disclosing vulnerabilities](#disclosing-vulnerabilities) | ||
- [Code Style](#code-style) | ||
- [Developer’s Certificate of Origin](#developers-certificate-of-origin) | ||
- [Pull request procedure](#pull-request-procedure) | ||
- [Communication](#communication) | ||
- [Conduct](#conduct) | ||
|
@@ -17,26 +16,32 @@ | |
|
||
## Introduction | ||
|
||
We welcome and encourage community contributions to ORY Oathkeeper. | ||
Please note: We take ORY Oathkeeper's security and our users' trust very seriously. If you believe you have found a | ||
security issue in ORY Oathkeeper, please responsibly disclose by contacting us at [email protected]. | ||
|
||
Since the project is still unstable, there are specific priorities for development. Pull requests that do not address these priorities will not be accepted until ORY Oathkeeper is production ready. | ||
First: if you're unsure or afraid of anything, just ask or submit the issue or pull request anyways. You won't be | ||
yelled at for giving it your best effort. The worst that can happen is that you'll be politely asked to change | ||
something. We appreciate any sort of contributions, and don't want a wall of rules to get in the way of that. | ||
|
||
Please familiarize yourself with the Contribution Guidelines and Project Roadmap before contributing. | ||
|
||
There are many ways to help ORY Oathkeeper besides contributing code: | ||
|
||
- Fix bugs or file issues | ||
- Improve the documentation | ||
That said, if you want to ensure that a pull request is likely to be merged, talk to us! You can find out our thoughts | ||
and ensure that your contribution won't clash or be obviated by ORY Oathkeeper's normal direction. A great way to do this is via | ||
the [ORY Community](https://community.ory.sh/) or join the [ORY Chat](https://www.ory.sh/chat). | ||
|
||
## Contributing Code | ||
|
||
Unless you are fixing a known bug, we **strongly** recommend discussing it with the core team via a GitHub issue or [in our chat](https://www.ory.sh/chat) before getting started to ensure your work is consistent with ORY Oathkeeper's roadmap and architecture. | ||
Unless you are fixing a known bug, we **strongly** recommend discussing it with the core team via a GitHub issue or | ||
[in our chat](https://www.ory.sh/chat) before getting started to ensure your work is consistent with | ||
ORY Oathkeeper's roadmap and architecture. | ||
|
||
All contributions are made via pull request. Note that **all patches from all contributors get reviewed**. After a pull request is made other contributors will offer feedback, and if the patch passes review a maintainer will accept it with a comment. When pull requests fail testing, authors are expected to update their pull requests to address the failures until the tests pass and the pull request merges successfully. | ||
All contributions are made via pull request. Note that **all patches from all contributors get reviewed**. After a pull | ||
request is made other contributors will offer feedback, and if the patch passes review a maintainer will accept it with | ||
a comment. When pull requests fail testing, authors are expected to update their pull requests to address the failures | ||
until the tests pass and the pull request merges successfully. | ||
|
||
At least one review from a maintainer is required for all patches (even patches from maintainers). | ||
|
||
Reviewers should leave a "LGTM" comment once they are satisfied with the patch. If the patch was submitted by a maintainer with write access, the pull request should be merged by the submitter after review. | ||
Reviewers should leave a "LGTM" comment once they are satisfied with the patch. If the patch was submitted by a | ||
maintainer with write access, the pull request should be merged by the submitter after review. | ||
|
||
## Disclosing vulnerabilities | ||
|
||
|
@@ -48,85 +53,41 @@ Please follow these guidelines when formatting source code: | |
|
||
* Go code should match the output of `gofmt -s` | ||
|
||
## Developer’s Certificate of Origin | ||
|
||
All contributions must include acceptance of the DCO: | ||
|
||
```text | ||
Developer Certificate of Origin | ||
Version 1.1 | ||
Copyright (C) 2004, 2006 The Linux Foundation and its contributors. | ||
660 York Street, Suite 102, | ||
San Francisco, CA 94110 USA | ||
Everyone is permitted to copy and distribute verbatim copies of this | ||
license document, but changing it is not allowed. | ||
Developer's Certificate of Origin 1.1 | ||
By making a contribution to this project, I certify that: | ||
(a) The contribution was created in whole or in part by me and I | ||
have the right to submit it under the open source license | ||
indicated in the file; or | ||
(b) The contribution is based upon previous work that, to the best | ||
of my knowledge, is covered under an appropriate open source | ||
license and I have the right under that license to submit that | ||
work with modifications, whether created in whole or in part | ||
by me, under the same open source license (unless I am | ||
permitted to submit under a different license), as indicated | ||
in the file; or | ||
(c) The contribution was provided directly to me by some other | ||
person who certified (a), (b) or (c) and I have not modified | ||
it. | ||
(d) I understand and agree that this project and the contribution | ||
are public and that a record of the contribution (including all | ||
personal information I submit with it, including my sign-off) is | ||
maintained indefinitely and may be redistributed consistent with | ||
this project or the open source license(s) involved. | ||
``` | ||
|
||
To accept the DCO, simply add this line to each commit message with your name and email address (`git commit -s` will do this for you): | ||
|
||
```text | ||
Signed-off-by: Jane Example <[email protected]> | ||
``` | ||
|
||
For legal reasons, no anonymous or pseudonymous contributions are accepted ([contact us](mailto:[email protected]) if this is an issue). | ||
|
||
## Pull request procedure | ||
|
||
To make a pull request, you will need a GitHub account; if you are unclear on this process, see GitHub's documentation on [forking](https://help.github.com/articles/fork-a-repo) and [pull requests](https://help.github.com/articles/using-pull-requests). Pull requests should be targeted at the `master` branch. Before creating a pull request, go through this checklist: | ||
To make a pull request, you will need a GitHub account; if you are unclear on this process, see GitHub's | ||
documentation on [forking](https://help.github.com/articles/fork-a-repo) and [pull requests](https://help.github.com/articles/using-pull-requests). | ||
Pull requests should be targeted at the `master` branch. Before creating a pull request, go through this checklist: | ||
|
||
1. Create a feature branch off of `master` so that changes do not get mixed up. | ||
1. [Rebase](http://git-scm.com/book/en/Git-Branching-Rebasing) your local changes against the `master` branch. | ||
1. Run the full project test suite with the `go test ./...` (or equivalent) command and confirm that it passes. | ||
1. Run `gofmt -s` (if the project is written in Go). | ||
1. Accept the Developer's Certificate of Origin on all commits (see above). | ||
1. Ensure that each commit has a subsystem prefix (ex: `controller: `). | ||
|
||
Pull requests will be treated as "review requests," and maintainers will give feedback on the style and substance of the patch. | ||
|
||
Normally, all pull requests must include tests that test your change. Occasionally, a change will be very difficult to test for. In those cases, please include a note in your commit message explaining why. | ||
Normally, all pull requests must include tests that test your change. Occasionally, a change will | ||
be very difficult to test for. In those cases, please include a note in your commit message explaining why. | ||
|
||
## Communication | ||
|
||
We use [discord](https://www.ory.sh/chat). You are welcome to drop in and ask questions, discuss bugs, etc. | ||
|
||
## Conduct | ||
|
||
Whether you are a regular contributor or a newcomer, we care about making this community a safe place for you and we've got your back. | ||
Whether you are a regular contributor or a newcomer, we care about making this community a safe place for you and | ||
we've got your back. | ||
|
||
* We are committed to providing a friendly, safe and welcoming environment for all, regardless of gender, sexual orientation, disability, ethnicity, religion, or similar personal characteristic. | ||
* We are committed to providing a friendly, safe and welcoming environment for all, regardless of gender, | ||
sexual orientation, disability, ethnicity, religion, or similar personal characteristic. | ||
* Please avoid using nicknames that might detract from a friendly, safe and welcoming environment for all. | ||
* Be kind and courteous. There is no need to be mean or rude. | ||
* We will exclude you from interaction if you insult, demean or harass anyone. In particular, we do not tolerate behavior that excludes people in socially marginalized groups. | ||
* Private harassment is also unacceptable. No matter who you are, if you feel you have been or are being harassed or made uncomfortable by a community member, please contact one of the channel ops or a member of the ORY Oathkeeper core team immediately. | ||
* We will exclude you from interaction if you insult, demean or harass anyone. In particular, we do not tolerate | ||
behavior that excludes people in socially marginalized groups. | ||
* Private harassment is also unacceptable. No matter who you are, if you feel you have been or are being harassed or | ||
made uncomfortable by a community member, please contact one of the channel ops or a member of the ORY | ||
Oathkeeper core team immediately. | ||
* Likewise any spamming, trolling, flaming, baiting or other attention-stealing behaviour is not welcome. | ||
|
||
We welcome discussion about creating a welcoming, safe, and productive environment for the community. If you have any questions, feedback, or concerns [please let us know](https://www.ory.sh/chat). |