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

318 migrate build system to poetry #319

Merged
merged 10 commits into from
Jul 28, 2022
Merged

Conversation

tensionhead
Copy link
Contributor

Replace setup.py with poetry

After installing poetry the following steps create a virtual env from the pyproject.toml:

# not sure what is the best way to actually get such python executable
# using conda definetely is an option..
poetry env use /path/to/python3.8
 
# grabs the packages and installs
poetry install

# activate poetry env
poetry shell

With this the syncopy environment was set up and the current shell runs within the virtual environment. As we have ipython in the dev dependencies, firing up this interpreter and import syncopy as spy should already work. Now with poetry build and poetry publish we are in principle already done!

Author Guidelines

  • Is the change set < 600 lines?

tensionhead and others added 10 commits July 26, 2022 17:09
- build and upload to test.pypi.org worked
- poetry is a bit fiddely with the used python version (but only acme pins with <3.9)
- TODO: test pip install from the test.pypi.org index

Changes to be committed:
	deleted:    conda2pip.py
	new file:   poetry.lock
	modified:   pyproject.toml
	deleted:    setup.cfg
- added readme, homepage, pypi classifiers and packaged the license
- apparently poetry publish fails silently if pypi does not like the
version string
@tensionhead tensionhead merged commit d91d47a into dev Jul 28, 2022
@tensionhead tensionhead deleted the 318-migrate-build-system-to-poetry branch July 28, 2022 12:38
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.

1 participant