Skip to content

Include optional packages in pyproject.toml and conda environments #39944

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

Merged
merged 8 commits into from
Jul 6, 2025

Conversation

tobiasdiez
Copy link
Contributor

@tobiasdiez tobiasdiez commented Apr 15, 2025

Adds the missing information about optional dependencies in the pyproject.toml and use this updated info to include these packages also in the conda environment.

📝 Checklist

  • The title is concise and informative.
  • The description explains in detail what this PR is about.
  • I have linked a relevant issue or discussion.
  • I have created tests covering the changes.
  • I have updated the documentation and checked the documentation preview.

⌛ Dependencies

Copy link

github-actions bot commented Apr 15, 2025

Documentation preview for this PR (built with commit b2a67f4; changes) is ready! 🎉
This preview will update shortly after each push to this PR.

@tobiasdiez
Copy link
Contributor Author

Sorry, this is not yet quiet ready. There are a bunch of failing tests. Didn't yet have the time to investigate where the are coming from.

@tobiasdiez
Copy link
Contributor Author

@isuruf This is now ready for review (the failing tests are just the usual, known random failures)

@tobiasdiez
Copy link
Contributor Author

@isuruf I'm setting this now to positive review based on your previous review. Please let me know if that's not okay with you or if you have any further remarks/suggestions for improvement!

@vbraun vbraun merged commit 85d6f2d into sagemath:develop Jul 6, 2025
22 of 24 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.

3 participants