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 defaults, update build process and add CI #16

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

adrien-berchet
Copy link
Contributor

@adrien-berchet adrien-berchet commented Nov 14, 2024

The defaults can not be updated because they are included in the function definitions. This new way allows to change them dynamically.

I also updated the setup.py, pyproject.toml and tox.ini files.

@bgeron

@bgeron
Copy link
Owner

bgeron commented Nov 29, 2024

Thank you, this is great!

@bgeron bgeron merged commit 0f96b37 into bgeron:main Nov 29, 2024
@bgeron
Copy link
Owner

bgeron commented Nov 29, 2024

I would make another release except I'm running into pypa/setuptools#4759 which is blocking the upload to PyPI.

I'll wait until that is fixed. Feel free to poke me once it's fixed and I'll go through the ceremony again.

@adrien-berchet
Copy link
Contributor Author

Hi @bgeron !
Thanks for the feedback, I will try to fix this soon.

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