Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

E20* - Incorrect number of whitespace #1306

Merged
merged 1 commit into from
Apr 2, 2018
Merged

Conversation

guedou
Copy link
Member

@guedou guedou commented Apr 1, 2018

Third PR of the #1277 serie. It fixes E20* aka incorrect number of whitespace. Automatic checks are enforced using tox.

Half of the errors reported by flake8 are now fixed.

Co-authored-by:  Guillaume Valadon <guillaume@valadon.net>
Co-authored-by:  Phil <phil@secdev.org>
Co-authored-by:  mtu <maxence.tury@ssi.gouv.fr>
Co-authored-by:  gpotter2 <gabriel@potter.fr>
Co-authored-by:  FlUxIuS <none@none>
Co-authored-by:  Pierre LALET <pierre.lalet@cea.fr>
Co-authored-by:  Mike Ryan <mikeryan@lacklustre.net>
Co-authored-by:  zer0@platinum <zer0@platinum>
Co-authored-by:  Lucas Pascal <lpascal@quarkslab.com>
Co-authored-by:  Victor Pfautz <v.pfautz@gmx.net>
Co-authored-by:  Pierre Lorinquer <pierre.lorinquer@ssi.gouv.fr>
Co-authored-by:  Taylor Trabun <ttrabun@isecpartners.com>
Co-authored-by:  Robin Jarry <robin.jarry@6wind.com>
Co-authored-by:  Jan Sebechlebsky <sebechlebskyjan@gmail.com>
Co-authored-by:  Klement Sekera <ksekera@cisco.com>
Co-authored-by:  Kawai, Hiroaki <hiroaki.kawai@gmail.com>
Co-authored-by:  Marcel Patzlaff <mpatzlaff@benocs.com>
Co-authored-by:  Jochen Bartl <jochen.bartl@gmail.com>
Co-authored-by:  Sabrina Dubroca <sd@queasysnail.net>
Co-authored-by:  bartex80 <piotr.bart@gmail.com>
Co-authored-by:  Dimitris Aragiorgis <dimara@grnet.gr>
Co-authored-by:  Martijn Thé <martijnthe@users.noreply.github.com>
Co-authored-by:  Todd Freed <todfreed@amazon.com>
Co-authored-by:  Walinton Cambronero <walinton.cambronero@hpe.com>
Co-authored-by:  haozheng10 <haozheng10@gmail.com>
Co-authored-by:  rdubourguais <renaud.dubourguais@synacktiv.com>
Co-authored-by:  sachetmittal <none@none>
Co-authored-by:  AStotal <astotal@gmail.com>
Co-authored-by:  Adam Karpierz <akarpie@ra.rockwell.com>
Co-authored-by:  Ajax <commial@gmail.com>
Co-authored-by:  Haim Daniel <haim@ma-gen.com>
Co-authored-by:  Nhoutain <houtainicolas@gmail.com>
Co-authored-by:  Robin Jarry <robin.jarry@6wind.com>
Co-authored-by:  mileswdavis@gmail.com <mileswdavis@gmail.com>
Co-authored-by:  ngkokkee <ngkokkee@hotmail.com>
Co-authored-by:  rbu@localhost.localdomain <rbu@localhost.localdomain>
@guedou guedou added the PEPin Apply PEP08 rules label Apr 1, 2018
@guedou guedou requested a review from p-l- April 1, 2018 09:07
@codecov-io
Copy link

codecov-io commented Apr 1, 2018

Codecov Report

Merging #1306 into master will increase coverage by 0.01%.
The diff coverage is 98.46%.

@@            Coverage Diff             @@
##           master    #1306      +/-   ##
==========================================
+ Coverage    84.9%   84.91%   +0.01%     
==========================================
  Files         161      161              
  Lines       38386    38386              
==========================================
+ Hits        32593    32597       +4     
+ Misses       5793     5789       -4
Impacted Files Coverage Δ
scapy/asn1/mib.py 90.14% <ø> (ø) ⬆️
scapy/contrib/openflow3.py 84.7% <ø> (ø) ⬆️
scapy/layers/tls/automaton_cli.py 77.02% <ø> (ø) ⬆️
scapy/contrib/isis.py 88.88% <ø> (ø) ⬆️
scapy/contrib/ripng.py 100% <ø> (ø) ⬆️
scapy/layers/tls/crypto/pkcs1.py 74.26% <ø> (ø) ⬆️
scapy/modules/p0f.py 65.37% <0%> (ø) ⬆️
scapy/contrib/ppi_geotag.py 89.96% <100%> (ø) ⬆️
scapy/automaton.py 82.22% <100%> (ø) ⬆️
scapy/layers/vrrp.py 84.09% <100%> (ø) ⬆️
... and 76 more

@p-l- p-l- merged commit 6861a35 into secdev:master Apr 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PEPin Apply PEP08 rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants