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

Unify requirements in pyproject file #26

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

ghisvail
Copy link
Contributor

@ghisvail ghisvail commented Apr 25, 2024

This PR removes the redundancy between the modern pyproject metadata and the legacy requirements files.

CI and RTD are also adjusted to make use of the pyproject metadata.

I have also dropped all version constraints. Unless there is a very good reason for it (like the use of a new symbol not present in earlier versions), they introduce more problems that they solve for downstream consumption by other projects.

Addresses some issues found in #25

@ghisvail ghisvail marked this pull request as ready for review April 25, 2024 12:45
@ghisvail ghisvail mentioned this pull request Apr 25, 2024
@ghisvail
Copy link
Contributor Author

@scossin let me know what you think of these changes. Meanwhile, I made iamsystem optional in medkit to allow installs on Python < 3.12.

@scossin scossin merged commit 04a4be7 into scossin:main Apr 30, 2024
6 checks passed
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