Skip to content

Conversation

@shoyer
Copy link

@shoyer shoyer commented Apr 4, 2018

lint_py2 and lint_py3 are no longer defined in the tox.ini config file. When
I try the currently suggested commands, I get the following error message:

ERROR: unknown environment 'lint_py2'

py27-lint and py3-lint appear to be the replacements.

lint_py2 and lint_py3 are no longer defined in the tox.ini config file. When
I try the currently suggested commands, I get the following error message:

  ERROR: unknown environment 'lint_py2'

py27-lint and py3-lint appear to be the replacements.
@angoenka
Copy link

angoenka commented Apr 4, 2018

Thanks for fixing this.

@angoenka
Copy link

angoenka commented Apr 4, 2018

@committer Can you please merge this.

@udim
Copy link
Member

udim commented Apr 6, 2018

CC: @melap

@melap
Copy link

melap commented Apr 6, 2018

We have a broken link unrelated to this PR that will not allow merging until it's fixed, I will merge this as soon as that's merged. (#413)

@melap
Copy link

melap commented Apr 6, 2018

It looks like #412 by @udim removes this section entirely, which one of these PRs should be the end goal?

@udim
Copy link
Member

udim commented Apr 6, 2018

#412 replaces the tox command line with a more stable gradlew one (which also takes care of setting up a virtualenv).

@melap
Copy link

melap commented Apr 6, 2018

ok, does that obsolete this PR then?

@udim
Copy link
Member

udim commented Apr 6, 2018

Yes, IMO.

@shoyer
Copy link
Author

shoyer commented Apr 6, 2018

agreed, this seems obsolete. (I just tested the gradlew command and it worked!)

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.

4 participants