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

tox -e pylint fails on home-assistant latest dev #28342

Closed
tulindo opened this issue Oct 30, 2019 · 2 comments · Fixed by #28403
Closed

tox -e pylint fails on home-assistant latest dev #28342

tulindo opened this issue Oct 30, 2019 · 2 comments · Fixed by #28403

Comments

@tulindo
Copy link
Contributor

tulindo commented Oct 30, 2019

Home Assistant release with the issue:
latest dev version

Last working Home Assistant release (if known):

Operating environment (Hass.io/Docker/Windows/etc.):

Integration:

Description of problem:
Looks like #28149 breaks local pylint test.

running on my dev enviromnent tox -e pylint
I'm getting:

ERROR: pylint: unresolvable substitution(s): 'PYLINT_ARGS'. Environment variables are missing or defined recursively.

Manually reverting the PR problem disappear.

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):


Additional information:

@tulindo tulindo changed the title tox -e pylint fails tox -e pylint fails on home-assistant latest dev Oct 30, 2019
@tulindo
Copy link
Contributor Author

tulindo commented Oct 30, 2019

@MartinHjelmare @balloob and @scop sorry to bother you. Do you think this is a problem of my development environment or something introduced with that PR?

@scop
Copy link
Member

scop commented Oct 31, 2019

My bad, sorry. Fix in #28403

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 a pull request may close this issue.

2 participants