style: add 'complexity' lint group to ruff config

[danieleades]

adds the 'complexity' lint group to the ruff config.

I believe this brings the ruff config inline with the flake8 config, and therefore makes the flake8 linting redundant.

for reference, the flake8 config is:

[flake8]
ignore = E203, E501, W503, B950
max-line-length = 88
select = C,E,F,W,B
per-file-ignores=
    __init__.py:F401

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.27%. Comparing base (fa63677) to head (c6e5f6c).

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1523      +/-   ##
==========================================
- Coverage   97.34%   97.27%   -0.07%     
==========================================
  Files          48       48              
  Lines        4557     4557              
==========================================
- Hits         4436     4433       -3     
- Misses        121      124       +3     

Flag	Coverage Δ
unittests	97.27% <100.00%> (-0.07%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[danieleades]

I'm a little surprised to see the behaviour differs. Maybe I'm missing something

[yajo]

It says we have a couple of overly complex functions. However both seem ok to me. could you please configure ruff to allow them, or add exclusion comments?

[danieleades]

It says we have a couple of overly complex functions. However both seem ok to me. could you please configure ruff to allow them, or add exclusion comments?

can do.
To be fair, I agree with Ruff's assessment that these functions are overly-complex and in need of a refactor. But i find it surprising that it appears to calculate the complexity differently to flake8

[sisp]

LGTM! 🎉 Thanks, @danieleades! 🙏