Contribution Checklist#
Purpose. Details to be considered before a pull request can be merged are summarized, primarily, to support by providing an overview and including references or code snippets. Secondarily, because the listings will not be duplicated, it serves as a reference for planning efforts or reviewing pull requests.
Change Preparation#
Every update shall enter via pull requests while each is linked to at least one
ticket (resolves #42, relates to #42 or similar).
SMALL PR
deviation
minors direct
exceptions (ticket templates, deployment CI)
Developers with repo access can use
dev/branch_start.shto start working on a new branch anddev/branch_close.shto initiate the PR.branches must use the pattern
<issue-number>_<summary>withsummaryin snake_case. Example: “42_compute_answer”.
Developers without repo access… let me know which details are missing here.
Code Changes#
ruff check,ruff formatunit test coverage (see [[APPXC Test Strategy]])
ToDo remarks for any known open topics (including planning tickets and referenced in ToDos
"ToDo #42")Ticket concluding remarks
Planning follow-up tickets
Localization: MO/PO files are updated (all translations available)
Functions are documented
How much depends on exposure
Minimum for public functions: Purpose (summary) and type annotation (including returns)
Features on Application Level#
If a feature is visible for final applications, it needs:
documentation (updates) in the feature section
automated BDD testing for behavior (application harness)
manual test for user experience (application harness GUI)
reference usage
documentation
testing the exact code in the testing
All bullet points below since it is always also a feature for application developers