Skip to content
Snippets Groups Projects

allow CI to fail if ruff or black is unhappy, and format source to make it pass

Merged Hans Fangohr requested to merge activate-style-checks-in-ci into linux-debian11
All threads resolved!

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Hans Fangohr added 2 commits

    added 2 commits

    • 6478fd09 - remove repeated import
    • 949e8fe8 - line-length editing to make ruff and black happy

    Compare with previous version

  • Hans Fangohr mentioned in commit ae162c17

    mentioned in commit ae162c17

  • Hans Fangohr added 2 commits

    added 2 commits

    • ae162c17 - execute all tests (functionality and style) at the same time
    • c8078b35 - white space tidy up

    Compare with previous version

  • Hans Fangohr added 2 commits

    added 2 commits

    Compare with previous version

  • Hans Fangohr marked this merge request as ready

    marked this merge request as ready

  • Hans Fangohr changed title from Draft: allow CI to fail if ruff or black is unhappy to allow CI to fail if ruff or black is unhappy, and format source to make it pass

    changed title from Draft: allow CI to fail if ruff or black is unhappy to allow CI to fail if ruff or black is unhappy, and format source to make it pass

  • I have run the tests manually on main and on this branch, and all 5 tests pass in both cases.

    So I think this is ready for review/merging.

    @akarnad

  • Hans Fangohr requested review from @akarnad

    requested review from @akarnad

  • Ashwin Kumar Karnad resolved all threads

    resolved all threads

  • mentioned in commit d5465c9d

  • Please register or sign in to reply
    Loading