remove ineffective checks against python2

[codecov-commenter]

Codecov Report

Merging #462 (ece9a52) into master (c210d67) will increase coverage by 0.02%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #462      +/-   ##
==========================================
+ Coverage   87.41%   87.43%   +0.02%     
==========================================
  Files          50       50              
  Lines        4330     4321       -9     
==========================================
- Hits         3785     3778       -7     
+ Misses        545      543       -2     

Impacted Files	Coverage Δ
bluepyopt/ephys/serializer.py	100.00% <ø> (ø)
bluepyopt/ephys/models.py	87.79% <100.00%> (+0.40%)	:arrow_up:

[anilbey]

@wvangeit I think only you have the authority to merge

[wvangeit]

Is there any specific reason to remove these lines, apart from readability? I don't really feel like breaking compatibility just for the sake of breaking compatibility. I'm just worried there might be some users using notebooks on old systems etc.

[anilbey]

No, only readability and a bit of confusion :) E.g. from __future__ is not the future but the past now

[wvangeit]

Ok, but I don't consider that enough arguments to possibly break external user's scripts.

[anilbey]

Ok, sure. I'm closing.