feat: allow cleaning of cancelled and aborted jobs

[katlyn]

Adds additional parameters to the builtin remove_old_jobs task to allow removing cancelled and aborted jobs in addition to failed and succeeded. Closes #1145.

Successful PR Checklist:

• [x] Tests
  • [ ] (not applicable?)
• [x] Documentation
  • [ ] (not applicable?)

PR label(s):

• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20breaking%20%F0%9F%92%A5
• [x] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20feature%20%E2%AD%90%EF%B8%8F
• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20bugfix%20%F0%9F%95%B5%EF%B8%8F
• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20miscellaneous%20%F0%9F%91%BE
• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20dependencies%20%F0%9F%A4%96
• [ ] https://github.com/procrastinate-org/procrastinate/labels/PR%20type%3A%20documentation%20%F0%9F%93%9A

[github-actions[bot]]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
procrastinate
manager.py
Project Total

This report was generated by python-coverage-comment-action

[medihack]

Great, thanks a lot for the contribution! LGTM.

@ewjoachim Do you have something to add? Otherwise, I will merge it into the main branch and make a minor release. Also, how about renaming remove_error resp. include_error to remove_failed resp. include_failed for the v3 release? Would look a bit more consistent to me.

[ewjoachim]

Also, how about renaming remove_error resp. include_error to remove_failed resp. include_failed for the v3 release? Would look a bit more consistent to me.

Yup.

Also, we need to be more consistent in the docstrings regarding the argument types. I believe they're not needed and sphinx can read them from type annotations, but we should make sure of that and fix it everywhere.