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

PEP8 error (Unused import) corrected #609

Merged
merged 1 commit into from Oct 12, 2017

Conversation

Projects
None yet
3 participants
@debojitkakoti
Contributor

debojitkakoti commented Oct 12, 2017

Signed-off-by: debojitkakoti debojkakoti88@gmail.com

What this PR does / why we need it: flake8 has been integrated into .travis.yml . It is throwing error of unused module, this has been corrected on this PR

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes # #607

Special notes for your reviewer:
This is regarding #607

PEP8 error (Unused import) corrected
Signed-off-by: debojitkakoti <debojkakoti88@gmail.com>
@debojitkakoti

This comment has been minimized.

Show comment
Hide comment
@debojitkakoti

debojitkakoti Oct 12, 2017

Contributor

@vharsh This is to correct the generate-inventory.py PEP8 error, as mentioned by you on #607 . Thanks!

Contributor

debojitkakoti commented Oct 12, 2017

@vharsh This is to correct the generate-inventory.py PEP8 error, as mentioned by you on #607 . Thanks!

@vharsh vharsh requested a review from ksatchit Oct 12, 2017

@kmova kmova requested review from ksatchit and removed request for ksatchit Oct 12, 2017

@ksatchit

LGTM

@vharsh

This comment has been minimized.

Show comment
Hide comment
@vharsh

vharsh Oct 12, 2017

Member

LGTM 👍

Member

vharsh commented Oct 12, 2017

LGTM 👍

@vharsh vharsh merged commit a0911a6 into openebs:master Oct 12, 2017

@debojitkakoti debojitkakoti deleted the debojitkakoti:pep8error branch Oct 12, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment