1
0
Fork 0
mirror of https://github.com/corona-warn-app/cwa-documentation synced 2025-07-03 11:58:29 +02:00

fix: linting contributing according

Signed-off-by: Johannes Amorosa <johannes.amorosa@endocode.com>
This commit is contained in:
Johannes Amorosa 2020-07-06 19:17:19 +02:00
parent 2f16aa4e4f
commit 95820d15bf
No known key found for this signature in database
GPG key ID: 94FDD520B1B6F31B
2 changed files with 58 additions and 21 deletions

View file

@ -9,6 +9,7 @@ Deutsche
e.g.
en_US
Gesundheitsamt
i.e.
Informationsfreiheit
Infrastrukturen
iOS
@ -19,7 +20,9 @@ Metadata
npm
Probenbegleitschein
pseudonymized
rebase
RPIs
sap.com
SARS-CoV-2
TalkBack
Telekom

View file

@ -6,11 +6,15 @@
## Code of Conduct
All members of the project community must abide by the [Contributor Covenant, version 2.0](CODE_OF_CONDUCT.md).
All members of the project community must abide by the
[Contributor Covenant, version 2.0](CODE_OF_CONDUCT.md).
Only by respecting each other we can develop a productive, collaborative community.
Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by contacting [corona-warn-app.opensource@sap.com](mailto:corona-warn-app.opensource@sap.com) and/or a project maintainer.
Instances of abusive, harassing, or otherwise unacceptable behavior may be
reported by contacting [corona-warn-app.opensource@sap.com](mailto:corona-warn-app.opensource@sap.com)
and/or a project maintainer.
We appreciate your courtesy of avoiding political questions here. Issues which are not related to the project itself will be closed by our community managers.
We appreciate your courtesy of avoiding political questions here. Issues which
are not related to the project itself will be closed by our community managers.
## Engaging in Our Project
@ -18,23 +22,34 @@ We use GitHub to manage reviews of pull requests.
* If you are a new contributor, see: [Steps to Contribute](#steps-to-contribute)
* If you have a trivial fix or improvement, go ahead and create a pull request, addressing (with `@...`) a suitable maintainer of this repository (see [CODEOWNERS](CODEOWNERS) of the repository you want to contribute to) in the description of the pull request.
* If you have a trivial fix or improvement, go ahead and create a pull request,
addressing (with `@...`) a suitable maintainer of this repository
(see [CODEOWNERS](CODEOWNERS) of the repository you want to contribute to) in
the description of the pull request.
* If you plan to do something more involved, please reach out to us and send an [email](mailto:corona-warn-app.opensource@sap.com). This will avoid unnecessary work and surely give you and us a good deal of inspiration.
* If you plan to do something more involved, please reach out to us and send an
[email](mailto:corona-warn-app.opensource@sap.com). This will avoid unnecessary
work and surely give you and us a good deal of inspiration.
* Relevant coding style guidelines are available in the respective sub-repositories as they are programming language-dependent.
* Relevant coding style guidelines are available in the respective
sub-repositories as they are programming language-dependent.
## Steps to Contribute
Should you wish to work on an issue, please claim it first by commenting on the GitHub issue that you want to work on. This is to prevent duplicated efforts from other contributors on the same issue.
Should you wish to work on an issue, please claim it first by commenting on the
GitHub issue that you want to work on. This is to prevent duplicated efforts
from other contributors on the same issue.
If you have questions about one of the issues, please comment on them, and one of the maintainers will clarify.
If you have questions about one of the issues, please comment on them, and one
of the maintainers will clarify.
We kindly ask you to follow the [Pull Request Checklist](#Pull-Request-Checklist) to ensure reviews can happen accordingly.
We kindly ask you to follow the [Pull Request Checklist](#Pull-Request-Checklist)
to ensure reviews can happen accordingly.
## Contributing Code
You are welcome to contribute code in order to fix a bug or to implement a new feature.
You are welcome to contribute code in order to fix a bug or to implement a new
feature.
The following rule governs code contributions:
@ -47,32 +62,51 @@ installation [document](INSTALL.md)
The following rule governs documentation contributions:
* Contributions must be licensed under the same license as code, the [Apache 2.0 License](LICENSE)
* Contributions must be licensed under the same license as code, the
[Apache 2.0 License](LICENSE)
## Pull Request Checklist
* Branch from the master branch and, if needed, rebase to the current master branch before submitting your pull request. If it doesn't merge cleanly with master, you may be asked to rebase your changes.
* Branch from the master branch and, if needed, rebase to the current master
branch before submitting your pull request. If it doesn't merge cleanly with
master, you may be asked to rebase your changes.
* Commits should be as small as possible while ensuring that each commit is correct independently (i.e., each commit should compile and pass tests).
* Commits should be as small as possible while ensuring that each commit is
correct independently (i.e., each commit should compile and pass tests).
* Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests. If tested manually, provide information about the test scope in the PR description (e.g. “Test passed: Upgrade version from 0.42 to 0.42.23.”).
* Test your changes as thoroughly as possible before you commit them. Preferably,
automate your test by unit/integration tests. If tested manually, provide
information about the test scope in the PR description (e.g. “Test passed:
Upgrade version from 0.42 to 0.42.23.”).
* Create _Work In Progress [WIP]_ pull requests only if you need clarification or an explicit review before you can continue your work item.
* Create _Work In Progress [WIP]_ pull requests only if you need clarification
or an explicit review before you can continue your work item.
* If your patch is not getting reviewed or you need a specific person to review it, you can @-reply a reviewer asking for a review in the pull request or a comment, or you can ask for a review by contacting us via [email](mailto:corona-warn-app.opensource@sap.com).
* If your patch is not getting reviewed or you need a specific person to review
it, you can @-reply a reviewer asking for a review in the pull request or a
comment, or you can ask for a review by contacting us via [email](mailto:corona-warn-app.opensource@sap.com).
* Post review:
* If a review requires you to change your commit(s), please test the changes again.
* If a review requires you to change your commit(s), please test the changes
again.
* Amend the affected commit(s) and force push onto your branch.
* Set respective comments in your GitHub review to resolved.
* Create a general PR comment to notify the reviewers that your amendments are ready for another round of review.
* Create a general PR comment to notify the reviewers that your amendments are
ready for another round of review.
## Issues and Planning
* We use GitHub issues to track bugs and enhancement requests.
* Please provide as much context as possible when you open an issue. The information you provide must be comprehensive enough to reproduce that issue for the assignee. Therefore, contributors should use but aren't restricted to the issue template provided by the project maintainers.
* Please provide as much context as possible when you open an issue. The
information you provide must be comprehensive enough to reproduce that issue for
the assignee. Therefore, contributors should use but aren't restricted to the
issue template provided by the project maintainers.
* When creating an issue, try using one of our issue templates which already contain some guidelines on which content is expected to process the issue most efficiently. If no template applies, you can of course also create an issue from scratch.
* When creating an issue, try using one of our issue templates which already
contain some guidelines on which content is expected to process the issue most
efficiently. If no template applies, you can of course also create an issue from
scratch.
* Please apply one or more applicable [labels](https://github.com/corona-warn-app/cwa-documentation/labels) to your issue so that all community members are able to cluster the issues better.
* Please apply one or more applicable [labels](https://github.com/corona-warn-app/cwa-documentation/labels)
to your issue so that all community members are able to cluster the issues better.