Skip to content

Latest commit

 

History

History
35 lines (27 loc) · 2.77 KB

CONTRIBUTING.md

File metadata and controls

35 lines (27 loc) · 2.77 KB
Raw

Reporting issues

  • Consider joining the chat for instant help.
  • Maybe your question is answered in the wiki somewhere. Search first!
  • Otherwise, just ask what you want to know. We're happy if we can help you and glad that you're using pmbootstrap!

Development

See pmbootstrap's Development Guide.

Contributing code changes

  • Fork this repository, commit your changes and then make a pull-request (PR).
  • Please test your code before submitting a PR.

Shell scripting

  • We don't write scripts for bash, but for busybox's ash shell, which is POSIX compliant (plus very few features from bash).
  • Use shellcheck to test your changes for issues before submitting. There is even an online version.
  • We're looking into automatizing this more, some files already get checked automatically by the static code analysis script.

Python

  • We use the PEP8 standard for Python code. Don't worry, you don't need to read all that, just run the autopep8 program on your changed code, and confirm with the static code analyis script that everything is PEP8 compliant. This script will run automatically on Travis CI when you make a change request, and it must pass for your code to get accepted.
  • We use the reST style for docstrings below functions (to comment what individual functions are doing, you'll see those when browsing through the code). Please stick to this format, and try to describe the important parameters and return values at least. Example from here:
"""
This is a reST style.

:param param1: this is a first param
:param param2: this is a second param
:returns: this is a description of what is returned
:raises keyError: raises an exception
"""
  • If it is feasible for you, try to run the testsuite on code that you have changed. The test/test_build.py case will build full cross-compilers for aarch64 and armhf, so it may take a long time. Testcases can be started with pytest and it's planned to run that automatically when making a new PR (see #64).

If you need any help, don't hesitate to open an issue and ask!