Skip to content

Latest commit

 

History

History
89 lines (70 loc) · 3.97 KB

CONTRIBUTING.md

File metadata and controls

89 lines (70 loc) · 3.97 KB

Contributing to OSO

Yay you are considering to contribute to OSO. 🙌

Everything written down here below are not strict rules but rather guidelines. Use your best judgement and if you have any suggestions about this document please feel free to submit a pull request.

Table of Contents

If you seek further information check out our Wiki.

Code of conduct

This project and everyone participating in it should obey to the OSO Code of Conduct. Please report unacceptable behavior to contact@ososystem.de

How can I contribute?

There are many ways to contribute, from writing tutorials, blog posts, improving the documentation, submitting bug reports and feature requests or writing code which can be incorporated into the project itself.

Code contribution

Choose an issue you want to do. Write in the comments that you are going to take care of it

Note: When working on an issue the issue number should be mentioned in the branch name e.g. #19-a-small-description. It is easier for us to track the changes this way.

try to add tests if they make sense and when you are done submit a pull request.

Bug reports

Have you seen a 🐛 around here? We don't like bugs.
Let's hunt them down together as good as we can! 🔪

  • Open up a bug issue
  • Describe it as good as you can and the following steps it took so others can reproduce it
  • Wait for someone to pick the issue up

And let's 🙏 that it won't show ever again!

Feature requests

Is there a feature you wish for which does not exist? No problem, just create a feature issue. Write down why you think it is useful and describe how it should work.

Styleguides

Code

Try to stick to the styleguide for the different languages which we have published in our wiki https://github.com/OSOSystem/oso-docs/wiki/Styleguide

Git commit messages

  • Use the present tense ("Add feature" not "Added feature").
  • Use the imperative mood ("Move cursor to..." not "Moves cursor to...").
  • When updating only the documentation, insert [ci skip] into the commit message so the build won't trigger.
  • Consider starting the commit message with an applicable emoji:
    • 🎨 :art: when improving the format/structure of the code
    • 🐎 :racehorse: when improving performance
    • 🚱 :non-potable_water: when plugging memory leaks
    • 📝 :memo: when writing docs
    • 🔧 :wrench: when fixing something
    • 🐛 :bug: when fixing a bug
    • 🔥 :fire: when removing code or files
    • 💚 :green_heart: when fixing the CI build
    • :white_check_mark: when adding tests
    • 🔒 :lock: when dealing with security
    • ⬆️ :arrow_up: when upgrading dependencies
    • ⬇️ :arrow_down: when downgrading dependencies
    • 👕 :shirt: when removing linter warnings
    • :heavy_plus_sign: when adding a feature
  • Consider adding a reference to an issue number

A commit could look like this e.g. :art: #19 Add delete method in help-requester service.

Code review process

When a pull request comes in, two of the maintainers have to approve it before it can be merged into the develop branch. If we notice an issue, we will write it down in the comments so you can fix it or discuss with us why we are wrong 😉

We will look at the pull requests on a regular basis.

Stay in touch

If you want to get in touch with us in a more relaxed atmosphere, consider joining the discord server.
💥Instant Transmission💥