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

E731 - do not assign a lambda expression, use a def #1588

Merged
merged 1 commit into from
Sep 12, 2018

Conversation

guedou
Copy link
Member

@guedou guedou commented Sep 8, 2018

Following discussions on #1277 this PR explicitly accept ignores E731.

@guedou guedou added the PEPin Apply PEP08 rules label Sep 8, 2018
@guedou
Copy link
Member Author

guedou commented Sep 11, 2018

@p-l- do you agree?

@p-l- p-l- merged commit d61084f into secdev:master Sep 12, 2018
@p-l-
Copy link
Member

p-l- commented Sep 12, 2018

@guedou yep!

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.

3 participants