- Make sure that you are listed in the AUTHORS file
- I have commented my code, particularly in hard-to-understand areas
- New and existing unit tests pass locally with my changes
- If this needs a change in OpenMS (OpenMS/OpenMS), the PR there is linked
No need to edit the CHANGELOG file: the maintainers update it for a release.
Click to expand
If your PR is failing you can check out- The details of the action statuses at the end of the PR or the "Checks" tab.
- The log of the failed job:
ctest --output-on-failureprints the output of each failed test.