generated from cisagov/ScubaGear
-
Notifications
You must be signed in to change notification settings - Fork 28
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Add PR template + CONTRIBUTING.md and update gitignore (#115)
* update gitignore * fix broken link * expand pylints reach * add contibuting.md * add pr template * add missing word
- Loading branch information
Showing
5 changed files
with
114 additions
and
4 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,49 @@ | ||
## 🗣 Description ## | ||
|
||
<!-- Describe the "what" of your changes in detail. --> | ||
|
||
### 💭 Motivation and context | ||
|
||
<!-- Why is this change required? --> | ||
<!-- What problem does this change solve? How did you solve it? --> | ||
<!-- Mention any related issue(s) here using appropriate keywords such --> | ||
<!-- as "closes" or "resolves" to auto-close them on merge. --> | ||
|
||
<!-- | ||
## 📷 Screenshots (if appropriate) ## | ||
Uncomment this section if a screenshot is needed. | ||
--> | ||
|
||
|
||
## 🧪 Testing | ||
|
||
<!-- How did you test your changes? How could someone else test this PR? --> | ||
<!-- Include details of your testing environment, and the tests you ran to --> | ||
<!-- see how your change affects other areas of the code, etc. --> | ||
|
||
## ✅ Pre-approval checklist ## | ||
|
||
<!-- Please read and check the boxes below as affirmation that this PR meets the requirements listed --> | ||
|
||
- [ ] This PR has an informative and human-readable title. | ||
- [ ] Changes are limited to a single goal - *eschew scope creep!* | ||
- [ ] If applicable, *All* future TODOs are captured in issues, which are referenced in the PR description. | ||
- [ ] The relevant issues PR resolves are linked preferably via [closing keywords](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword). | ||
- [ ] All relevant type-of-change labels have been added. | ||
- [ ] I have read and agree to the [CONTRIBUTING.md](https://github.com/cisagov/ScubaGoggles/blob/main/CONTRIBUTING.md) document. | ||
- [ ] These code changes follow [cisagov code standards](https://github.com/cisagov/development-guide). | ||
- [ ] All relevant repo and/or project documentation has been updated to reflect the changes in this PR. | ||
- [ ] Tests have been added and/or modified to cover the changes in this PR. | ||
- [ ] All new and existing tests pass. | ||
|
||
## ✅ Pre-merge Checklist | ||
|
||
- [ ] This PR has been smoke tested to ensure main is in a functional state when this PR is merged. | ||
- [ ] Squash all commits into one PR level commit using the `Squash and merge` button. | ||
|
||
## ✅ Post-merge Checklist | ||
|
||
- [ ] Delete the branch to clean up. | ||
- [ ] Close issues resolved by this PR if the [closing keywords](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword) did not activate. |
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 |
---|---|---|
@@ -0,0 +1,36 @@ | ||
# Welcome # | ||
|
||
We're so glad you're thinking about contributing to this open-source project! If you're unsure or hesitant to make a recommendation, just ask, submit the issue, or pull request. The worst that can happen is that you'll be politely asked to change something. We appreciate any sort of contribution(s), and don't want a wall of rules to stifle innovation. | ||
|
||
Before contributing, we encourage you to read our CONTRIBUTING policy (you are here), our [LICENSE](https://github.com/cisagov/ScubaGoggles/blob/main/LICENSE), and our [README](https://github.com/cisagov/ScubaGoggles/blob/main/README.md), all of which are in this repository. | ||
|
||
## Issues | ||
|
||
If you want to report a bug or request a new feature, the most direct method is to [create an issue](https://github.com/cisagov/ScubaGoggles/issues) in this repository. | ||
We recommend that you first search through existing issues (both open and closed) to check if your particular issue has already been reported. | ||
|
||
If it has then you might want to add a comment to the existing issue. | ||
|
||
If it hasn't then feel free to create a new one. | ||
|
||
Please follow the provided template and fill out all sections. | ||
We have a `Bug Report` and `Idea` Template. | ||
|
||
## Pull Requests (PR) | ||
|
||
If you choose to submit a pull request, it will be required to pass various linters and sanity checks in our continuous integration (CI) pipeline, before we merge it. Your pull request may fail these checks, and that's OK. If you want you can stop there and wait for us to make the necessary corrections to ensure your code passes the CI checks, you're more than within your rights; however, it helps our team greatly if you fix the issues found by our CI pipeline. | ||
|
||
### Quality assurance and code reviews | ||
|
||
All PRs will be tested, vetted, and reviewed by our team before being merged to the main branch. Please stand by to address questions, concerns, or improvement suggestions we may have about your PR. | ||
|
||
## Public domain | ||
|
||
This project is in the public domain within the United States, and | ||
copyright and related rights in the work worldwide are waived through | ||
the [CC0 1.0 Universal public domain | ||
dedication](https://creativecommons.org/publicdomain/zero/1.0/). | ||
|
||
All contributions to this project will be released under the CC0 | ||
dedication. By submitting a pull request, you are agreeing to comply | ||
with this waiver of copyright interest. |
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