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.

Sign up for GitHub

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

Jump to bottom

pep8 error on brave-core in master #9732

Closed
bsclifton opened this issue May 12, 2020 · 0 comments · Fixed by brave/brave-core#5522
Closed

pep8 error on brave-core in master #9732

bsclifton opened this issue May 12, 2020 · 0 comments · Fixed by brave/brave-core#5522
Assignees
@bsclifton
Labels
dev-concern QA/No release-notes/exclude
Milestone
1.10.x - Release

Comments

@bsclifton
Copy link
Member

Description

Python script has a lint (pep8) violation

Steps to Reproduce

  1. npm run pep8 in src/brave

Actual result:

c:\bb1\src\brave>npm run pep8

> brave-core@1.10.47 pep8 c:\bb1\src\brave
> pycodestyle --max-line-length 120 -r script

script\lib\env_util.py:27:17: E741 ambiguous variable name 'l'

Expected result:

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

@bsclifton bsclifton

Labels
dev-concern QA/No release-notes/exclude
Projects
None yet
Milestone
1.10.x - Release
Development

Successfully merging a pull request may close this issue.

Fix for pep8 violation brave/brave-core
1 participant
@bsclifton