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

Cookiecutter #172

Closed
wants to merge 1 commit into from
Closed

Conversation

pattonw
Copy link

@pattonw pattonw commented Jul 31, 2019

Add a reference to a mixed layout python/rust package cookiecutter.

Very much still a work in progress.

pip install . and deploying to pypi on tagged commit through travis_ci should work out of the box.

Most other features have not been tested.

Contributions welcome

Add a reference to a mixed layout python/rust package cookiecutter
@konstin
Copy link
Member

konstin commented Aug 5, 2019

I think that an example/template for the mixed layout is definitely a good idea, however I'm not sure if cookiecutter-pypackage is right base for that. Cookiecutter is centered around setup.py, while pyo3-pack is based on the new PEP 517 with its pyproject.toml. Also note that idiomatic testing with tox is currently blocked on a tox bug (tox-dev/tox#1344).

@pattonw
Copy link
Author

pattonw commented Aug 7, 2019

My fork of cookiecutter-pypackage has removed setup.py and tox. It just has a make pattern for testing which uses pyo3-pack develop and pytest. Travis handles making virtual environments for testing different versions.

@konstin konstin closed this Jan 23, 2020
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.

2 participants