Skip to content

Commit

Permalink
Update CONTRIBUTING.md
Browse files Browse the repository at this point in the history
  • Loading branch information
cornelinux authored Aug 21, 2019
1 parent 1defb0f commit 4d5b362
Showing 1 changed file with 7 additions and 12 deletions.
19 changes: 7 additions & 12 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ be really easy.

You are using privacyIDEA in your network? Tell about it!
Write a blog post, tell your friends or simply twitter about it.
This will make privacyIDEA wider known and attract new contributors.
This will make privacyIDEA wider known and attract new users and contributors.

## Talk to us

Expand All @@ -30,22 +30,17 @@ So you can make privacyIDEA better and earn karma points by
sharing your experience and helping other, younger users.

By doing so the developers have more time to develop and
improving your beloved project and hopefully allowing
the developers to earn some money to make a living.
improving your beloved project.

So **you are "the community"**, go and help others!

https://community.privacyidea.org

## Money
## Professional services

This is an important part to contribute to privacyIDEA.
To allow developers to work full time on privacyIDEA they
have to make money.

One possibility to do this and also get rid of the NO-WARRANTY
label is to get privacyIDEA Enterprise Edition including a
Service Level Agreement or corresponding consultancy.
If you want to get rid of the NO-WARRANTY
label take a look at privacyIDEA Enterprise Edition including a
Service Level Agreement or get professional consultancy:

- consultancy
https://netknights.it/en/leistungen/one-time-services/
Expand Down Expand Up @@ -223,4 +218,4 @@ If the reviewer approves a pull request, and if no other reviews have been reque

##### Multiple Reviewers

The developer may also request reviews from multiple team members. This makes sense if the PR spans across multiple architectural layers (e.g. backend *and* frontend), or if the developer wants both a functional (does the PR fix the bug?) and technical review (is the code okay?). If a developer requests multiple reviews, the PR description should explicitly state if *all* reviews should be positive, or if *one* positive review is sufficient.
The developer may also request reviews from multiple team members. This makes sense if the PR spans across multiple architectural layers (e.g. backend *and* frontend), or if the developer wants both a functional (does the PR fix the bug?) and technical review (is the code okay?). If a developer requests multiple reviews, the PR description should explicitly state if *all* reviews should be positive, or if *one* positive review is sufficient.

0 comments on commit 4d5b362

Please sign in to comment.