Merge branch 'develop' into main

Author: schmide

doc/guide/Reviewer_Checklist.md

@@ -5,6 +5,7 @@
 * The code should be easily readable (for infos e.g. here [here](https://treyhunner.com/2017/07/craft-your-python-like-poetry/?__s=jf8h91lx6zhl7vv6o9jo))
 * Variable names should be chosen to be clear. Avoid `item, element, var, list` etc... 
 * Avoid as much as possible hard-coded indices for list (no `x = l[0], y = l[1]`) (see also [here](https://treyhunner.com/2018/03/tuple-unpacking-improves-python-code-readability/))
+* Avoid mutable as default values for functions and methods.
 * Use pythonic loops, list comprehensions etc.
 * Make sure the unit test are testing all the relevant parts of the code
 * Check the docstring (is everything clearly explained, are the default values given an clear)

doc/guide/coding_conventions.rst

@@ -3,7 +3,7 @@
 Coding Conventions
 ==================
 
-Contributions are very welcome! But we need to keep a certain order. Please seriously consider the following guidelines.
+Contributions are very welcome! But we need to keep a certain order. Please earnestly consider the following guidelines.
 
 Unit Tests
 ----------
@@ -77,16 +77,6 @@ Cython, Numba, ...
 - When using Numba, make sure to avoid Python objects as, otherwise, Numba will
   use the less efficient `object mode <https://numba.pydata.org/numba-doc/latest/glossary.html#term-object-mode>`_.
 
-Exposed Interface
-=================
-Library functions???
-
 Configuration
 =============
-Dealing with constants
-----------------------
-??? central registry?
-
-Dealing with defaults
----------------------
-???
+- URLs of external resources and locations of data directories should always be defined in the config.py file and not declared as constants.