jazzband / django-fsm-log

Automatic logging for Django FSM
https://django-fsm-log.readthedocs.io/en/latest/
MIT License
242 stars 78 forks source link

Silent these warnings #174

Closed ticosax closed 1 year ago

ticosax commented 1 year ago
B018 Found useless expression. Either assign it to a variable or remove it.
codecov[bot] commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@ecf4c7f). Click here to learn what that means. The diff coverage is 100.00%.

:exclamation: Current head 6a4f69b differs from pull request most recent head a6f9c72. Consider uploading reports for the commit a6f9c72 to get more accurate results

@@            Coverage Diff            @@
##             master     #174   +/-   ##
=========================================
  Coverage          ?   95.45%           
=========================================
  Files             ?       24           
  Lines             ?      528           
  Branches          ?        0           
=========================================
  Hits              ?      504           
  Misses            ?       24           
  Partials          ?        0           
Impacted Files Coverage Δ
tests/test_model.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

ticosax commented 1 year ago

Why not, giving it to a variable was the other option.

Because it will raise another warning that the variable is not used