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

Add the flake8 to be the default code formatter #401

Merged
merged 4 commits into from
Jun 2, 2020
Merged

Conversation

wenbingl
Copy link
Member

@wenbingl wenbingl commented Jun 1, 2020

No description provided.

@lgtm-com
Copy link

lgtm-com bot commented Jun 1, 2020

This pull request fixes 1 alert when merging aa0c44b into 18c4dc1 - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Jun 1, 2020

This pull request fixes 1 alert when merging 8eed29b into 18c4dc1 - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Jun 1, 2020

This pull request introduces 1 alert and fixes 1 when merging 45b3488 into 18c4dc1 - view on LGTM.com

new alerts:

  • 1 for Unused import

fixed alerts:

  • 1 for 'import *' may pollute namespace

@lgtm-com
Copy link

lgtm-com bot commented Jun 2, 2020

This pull request fixes 1 alert when merging b87dc5f into 18c4dc1 - view on LGTM.com

fixed alerts:

  • 1 for 'import *' may pollute namespace

@wenbingl wenbingl requested review from xadupre and jiafatom June 2, 2020 00:21
@jiafatom jiafatom merged commit 57f8e15 into onnx:master Jun 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants