Skip to content

Conversation

mkoeppe
Copy link
Contributor

@mkoeppe mkoeppe commented May 1, 2023

📚 Description

📝 Checklist

  • The title is concise, informative, and self-explanatory.
  • 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 accordingly.

⌛ Dependencies

@tobiasdiez tobiasdiez added the s: run conda ci Run the conda workflow on this PR. label May 1, 2023
@github-actions
Copy link

github-actions bot commented May 1, 2023

Documentation preview for this PR is ready! 🎉
Built with commit: 58e947f

Copy link
Contributor

@tobiasdiez tobiasdiez left a comment

Choose a reason for hiding this comment

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

It's an improvement, but the optional workflows are still failing. Mainly because of open issues upstream, e.g.

Should we remove these packages from the optional environment for now?

@mkoeppe
Copy link
Contributor Author

mkoeppe commented May 1, 2023

Yes, let's do that. I've opened #35593 for it

@vbraun vbraun merged commit 4ae1898 into sagemath:develop May 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bootstrap-conda: Do not include sage in src/environment-optional.yml
3 participants