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
Fix #1236 typos in isolated.py build method #1235
Fix #1236 typos in isolated.py build method #1235
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a bug fix changelog please, otherwise looks good. We have some flit/poetry integration tests I wonder how those did not failed 🤔
@gaborbernat Sure, I added a changelog and also opened an issue :) |
docs/changelog.rst
Outdated
@@ -11,6 +11,15 @@ with advance notice in the **Deprecations** section of releases. | |||
|
|||
.. towncrier release notes start | |||
|
|||
v3.8.4 (2019-03-30) | |||
------------------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
eh, not here, see https://github.com/tox-dev/tox/blob/master/docs/changelog/examples.rst
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. fixed it
The failing test is not related to your PR, feel free to add a skip for it (python 2, windows only). |
@gaborbernat Thanks for guiding me! What is the right way to skip failing test? Should I make changes to |
pytest skip decorator 👌 |
Thanks! 🔥 |
Hi, there!
There are typos in tox/package/builder/isolated.py making it impossible to install package with requirements in pyproject.toml
I've been using tox (3.7.0) with flit as a build backend. After upgrading to tox 3.8 I keep getting an error whenever try to run tox.
It seems like during refactoring there have been made some typos. I made two commits: