Skip to content

Commit

Permalink
Merge pull request #121 from Bycelium/documentation/improve_contribute
Browse files Browse the repository at this point in the history
📝 Improve the contribute.md file
  • Loading branch information
MathisFederico authored Dec 19, 2021
2 parents bf84d29 + b4892ad commit c91586c
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

[![Pytest badge](https://github.com/Bycelium/OpenCodeBlocks/actions/workflows/python-tests.yml/badge.svg?branch=master)](https://github.com/MathisFederico/OpenCodeBlocks/actions/workflows/python-tests.yml) [![Pylint badge](https://img.shields.io/endpoint?url=https%3A%2F%2Fgist.githubusercontent.com%2FMathisFederico%2F00ce73155619a4544884ca6d251954b3%2Fraw%2Fopencodeblocks_pylint_badge.json)](https://github.com/MathisFederico/OpenCodeBlocks/actions/workflows/python-pylint.yml) [![Unit coverage badge](https://img.shields.io/endpoint?url=https%3A%2F%2Fgist.githubusercontent.com%2FMathisFederico%2F00ce73155619a4544884ca6d251954b3%2Fraw%2Fopencodeblocks_unit_coverage_badge.json)](https://github.com/MathisFederico/OpenCodeBlocks/actions/workflows/python-coverage.yml) [![Integration coverage badge](https://img.shields.io/endpoint?url=https%3A%2F%2Fgist.githubusercontent.com%2FMathisFederico%2F00ce73155619a4544884ca6d251954b3%2Fraw%2Fopencodeblocks_integration_coverage_badge.json)](https://github.com/MathisFederico/OpenCodeBlocks/actions/workflows/python-coverage.yml)

Whenever you encounter a :beetle: **bug** or have :tada: **feature request**,
Whenever you encounter a :beetle: **bug** or have :tada: **feature request**,
report this via [GitHub issues](https://github.com/Bycelium/PyFlow/issues).

We are happy to receive contributions in the form of **pull requests** via GitHub.
Expand All @@ -11,10 +11,10 @@ Feel free to fork the repository, implement your changes and create a merge requ

## Getting started

First, make sure you have `python` installed. You will need to install all the `requirements` and the `requirements-dev` using the following commands:
First, make sure you have `python` installed. You will need to install all the `requirements` and the `requirements-dev` using the following commands:

* `pip install -r requirements.txt`
* `pip install -r requirements-dev.txt`
* `pip install -r requirements.txt`
* `pip install -r requirements-dev.txt`

You can run the program with `python main.py`

Expand All @@ -24,10 +24,10 @@ Before doing your **pull request**, check using `pylint` and `pytest` that there
pylint .\opencodeblocks\
```

Some `pylint` issues can be fixed automatically using `autopep8`, with the following command:
Some `pylint` issues can be fixed automatically using `black`, with the following command:

```bash
autopep8 --in-place --recursive --aggressive opencodeblocks
black .
```

```bash
Expand All @@ -36,12 +36,13 @@ pytest --cov=opencodeblocks --cov-report=html tests/unit

We want to keep the *Pylint* score above *9.0*.

The comments and docstrings should preferably follow [these](https://google.github.io/styleguide/pyguide.html#38-comments-and-docstrings) guidelines.

## Git Commit Messages

Commits should start with a Capital letter and should be written in present tense (e.g. ``:tada: Add cool new feature`` instead of ``:tada: Added cool new feature``).
You should also start your commit message with one or two applicable emoji. This does not only look great but also makes you rethink what to add to a commit. Make many but small commits!


Emoji | Description
-----------------|-------------
:tada: `:tada:` | When you add a cool new feature
Expand Down

0 comments on commit c91586c

Please sign in to comment.