search

jazzband / django-analytical

Analytics services for Django projects
MIT License
1.2k stars 168 forks source link

Readability counts, code must be self-explanatory #209

Closed bittner closed 2 years ago

bittner commented 2 years ago

I feel ashamed that it took me 3 attempts to get it (probably) right. But I feel it was maybe worth it. PEP20 says it all.

codecov[bot] commented 2 years ago

Codecov Report

Merging #209 (f690a5f) into main (d82ca0a) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #209   +/-   ##
=======================================
  Coverage   94.52%   94.52%           
=======================================
  Files          32       32           
  Lines        1388     1388           
=======================================
  Hits         1312     1312           
  Misses         76       76

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d82ca0a...f690a5f. Read the comment docs.