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

Fix usedevelop without setup.py #2198

Closed

Conversation

AntoineD
Copy link
Contributor

@AntoineD AntoineD commented Sep 2, 2021

Fix #2197

I was able to check the fix on an external project but unfortunately the new test does not work when the cmd fixture is called the second time (error at teardown).
Please let me know how to fix this test or feel free to change it.

Copy link
Member

@gaborbernat gaborbernat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure how you even get that far.

Copy link
Member

@gaborbernat gaborbernat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you can make it work for both old and new world I'm happy to accept this PR.

@AntoineD
Copy link
Contributor Author

AntoineD commented Sep 2, 2021

What do you mean by old and new world?

@gaborbernat
Copy link
Member

It should work with any setuptools/pip version the user might use.

@AntoineD
Copy link
Contributor Author

AntoineD commented Sep 7, 2021

It seems to me that it does not worth the effort then, as compared to creating an almost empty setup.py.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

usedevelop without setup.py failure
2 participants