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

test no longer for sage 9.1, add test for sage 9.8 #89

Closed
wants to merge 6 commits into from

Conversation

fchapoton
Copy link
Contributor

Let us see if everything is fine with the latest sage.

Sage 9.1 is too old now.

@fchapoton
Copy link
Contributor Author

sadly, sage 9.8 is not yet available on docker hub

fchapoton and others added 5 commits April 3, 2023 11:29
trying to use ghcr.io docker images
change docker image
try again
try again
@NathanDunfield
Copy link
Member

@fchapoton First, many thanks for getting updated SageMath docker images created. As I followed that thread in sage-devel, I added 9.8 and 10.0 in 2bcf50b. Is ghcr.io/sagemath now the canonical location for these images?

@fchapoton
Copy link
Contributor Author

hello,
it seems you wrongly added the non-existing 9.10 instead of 10.0.
Concerning the location, I would suggest to stick to dockerhub
so my branch here ismaybe notso pertinent

@NathanDunfield
Copy link
Member

Thanks for catching the nonexistent 9.10 issue, I feel like the CI should have raised an error there but it didn't. I have corrected to 10.0, so will close this ticket.

@NathanDunfield
Copy link
Member

Ugh, I spoke too soon. I added 10.0, but GH actions "rounds" this to 10 and then sagemath/sagemath:10 does not exist.

@NathanDunfield
Copy link
Member

Ugh, I spoke too soon. I added 10.0, but GH actions "rounds" this to 10 and then sagemath/sagemath:10 does not exist.

I solved this problem after a fashion by referring to 10.0 as latest.

@fchapoton fchapoton deleted the test_on_sage_9.8 branch July 30, 2023 13:10
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