Skip to content

Conversation

@dimpase
Copy link
Member

@dimpase dimpase commented Jul 1, 2025

remove $ to print the version correctly, and remove unneeded "..."

the extra $ leads to the minimal version not being printed, and extra "..." around minimal version are confusing (and not needed)

📝 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

remove `$` to print the version correctly, and remove unneeded `"..."`
@dimpase
Copy link
Member Author

dimpase commented Jul 1, 2025

I discovered all this while figuring out why the trunk maxima (which is version 5.47foo by default) doesn't get recognised. Perhaps we should add a comment on this.

@dimpase dimpase requested a review from tobiasdiez July 1, 2025 18:16
@github-actions
Copy link

github-actions bot commented Jul 1, 2025

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

@vbraun vbraun merged commit 34e96ca into sagemath:develop Jul 6, 2025
34 checks passed
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.

3 participants