Unpin flake8 to allow v3

[yurishkuro]

and fix a few minor breaks

also remove tests from lint, because they were skipped in flake8 config anyway

[black-adder]

also remove tests from lint, because they were skipped in flake8 config anyway

I actually like linting tests... the whole codebase should conform; can we add a rule to flake8 config to run on tests?

[yurishkuro]

@black-adder I agree with you, however at the moment tests/ does not pass flake8 lint, and it is explicitly excluded from flake8 configuration in setup.cfg, so it was misleading to include the dir in the lint command.

[black-adder]

fair enough

[codecov]

Codecov Report

Merging #231 into master will increase coverage by 0.1%.
The diff coverage is 77.77%.

@@            Coverage Diff            @@
##           master     #231     +/-   ##
=========================================
+ Coverage    94.8%   94.91%   +0.1%     
=========================================
  Files          25       25             
  Lines        1848     1848             
  Branches      245      245             
=========================================
+ Hits         1752     1754      +2     
+ Misses         62       61      -1     
+ Partials       34       33      -1

Impacted Files	Coverage Δ
crossdock/server/serializer.py	85.5% <0%> (ø)	⬆️
jaeger_client/sampler.py	100% <100%> (ø)	⬆️
jaeger_client/config.py	91.42% <85.71%> (ø)	⬆️
jaeger_client/throttler.py	96.36% <0%> (+1.81%)	⬆️

---

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 24603de...93b5ea5. Read the comment docs.