Skip to content

Add needs tags for database_cremona_(mini_)ellcurve#42276

Draft
tobiasdiez wants to merge 1 commit into
sagemath:developfrom
tobiasdiez:cremona_ellcurve_feature
Draft

Add needs tags for database_cremona_(mini_)ellcurve#42276
tobiasdiez wants to merge 1 commit into
sagemath:developfrom
tobiasdiez:cremona_ellcurve_feature

Conversation

@tobiasdiez
Copy link
Copy Markdown
Contributor

@tobiasdiez tobiasdiez commented May 30, 2026

The uv-based CI workflows are failling on some systems with a quite a few errors due to a missing/not found ellcurve db (see eg https://github.com/cxzhong/sage/actions/runs/26278272434/job/77347800338#step:9:60). So we add here the required #needs tags.

There were quite a few places (around 1000), so this was done in a semi-automated way. I double-checked a few locations manually, but cannot guarantee that I got all instances nor that all annotations are 100% 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

@github-actions
Copy link
Copy Markdown

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

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.

1 participant