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

Add packaging with poetry #40

Merged
merged 1 commit into from
Nov 10, 2022
Merged

Add packaging with poetry #40

merged 1 commit into from
Nov 10, 2022

Conversation

sandrobonazzola
Copy link
Contributor

@sandrobonazzola sandrobonazzola commented Nov 7, 2022

Add packaging using poetry

Help installing the package as pip install git+https://github.com/distributed-system-analysis/smallfile.git@main
Signed-off-by: Sandro Bonazzola [email protected]

Copy link
Member

@portante portante left a comment

Choose a reason for hiding this comment

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

Let's also drop setup.cfg and requirements.txt (making sure they are both properly covered by pyproject.toml), and then update the README.md where necessary to describe the use of poetry.

@sandrobonazzola
Copy link
Contributor Author

Let's also drop setup.cfg and requirements.txt (making sure they are both properly covered by pyproject.toml), and then update the README.md where necessary to describe the use of poetry.

Ok for dropping requirements.txt but we can't remove setup.cfgbecause pycodestyle and flake8 doesn't accept configuration via pyproject.toml yet.

setup.cfg Outdated Show resolved Hide resolved
setup.cfg Outdated Show resolved Hide resolved
@sandrobonazzola sandrobonazzola self-assigned this Nov 9, 2022
Signed-off-by: Sandro Bonazzola <[email protected]>
@portante portante merged commit e6a31b1 into distributed-system-analysis:main Nov 10, 2022
@sandrobonazzola sandrobonazzola deleted the poetry-build branch November 10, 2022 07:50
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