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

Introduce Poetry and pytest #39

Merged
merged 1 commit into from
May 21, 2024
Merged

Conversation

jiak94
Copy link
Contributor

@jiak94 jiak94 commented May 21, 2024

This PR contains only engineering optimization. If you think this does not help, please close the PR.

Poetry

Poetry is a versatile tool for managing dependencies and Python versions. To begin development, simply run the command poetry install, which will install all required dependencies and create a virtual env for this package. To add a new dependency, use the command poetry add [package]. For more information, visit the Poetry documentation.

Github Actions

I set up two GitHub Actions: one for testing and one for releases.

The unittest action is triggered for every PR but skips the test_ob test because it takes too long to complete. When merging into master, the unittest action runs a full test suite.

The build_publish action simplifies releasing the package. It automatically builds and pushes the package to PyPI for every tag you create, using the tag name for versioning. Remember to set the PIPY_TOKEN in the repository secrets.

Pytest

I recreated those testcases in pytest to skip the test_ob

Thank you so much for creating this amazing package.

@joshyattridge joshyattridge merged commit 3922871 into joshyattridge:master May 21, 2024
@joshyattridge
Copy link
Owner

Hi @jiak94,

This all seems very helpful. thank you for this addition it will make my life easier!

Regards,
Josh

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