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.

Sign up for GitHub

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

Jump to bottom

Remove outdated (pre-PEP 517/518) practices in make check, fix ci-sage #190

Merged
merged 6 commits into from
Jun 9, 2024
Merged

Remove outdated (pre-PEP 517/518) practices in make check, fix ci-sage #190

merged 6 commits into from
Jun 9, 2024

Conversation

mkoeppe
Copy link

@mkoeppe mkoeppe commented Oct 7, 2023

No description provided.

@mkoeppe mkoeppe self-assigned this Oct 7, 2023
@mkoeppe mkoeppe requested review from dimpase and malb October 7, 2023 23:02
@mkoeppe mkoeppe changed the title CI: Fix ci-sage Remove outdated (pre-PEP 517/518) practices in make check, fix ci-sage Oct 8, 2023
@mkoeppe mkoeppe force-pushed the ci_sage_fixup branch 6 times, most recently from 14118e2 to 4f28a85 Compare October 8, 2023 02:07
@mkoeppe mkoeppe marked this pull request as draft October 8, 2023 03:33
@dimpase
Copy link
Member

dimpase commented Oct 8, 2023

seems to be the same story:

2023-10-08T02:43:17.5450390Z   [cysignals-git]     ERROR: Could not find a version that satisfies the requirement setuptools (from versions: none)
2023-10-08T02:43:17.5450681Z   [cysignals-git]     ERROR: No matching distribution found for setuptools
2023-10-08T02:43:17.5451248Z   [cysignals-git]     WARNING: There was an error checking the latest version of pip.
2023-10-08T02:43:17.5451499Z   [cysignals-git]     error: subprocess-exited-with-error

@mkoeppe mkoeppe force-pushed the ci_sage_fixup branch 3 times, most recently from 148eaf0 to 3c6a507 Compare June 9, 2024 05:48
@mkoeppe mkoeppe marked this pull request as ready for review June 9, 2024 06:58
@mkoeppe mkoeppe merged commit 30f9d2d into sagemath:main Jun 9, 2024
23 of 116 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dimpase dimpase Awaiting requested review from dimpase

@malb malb Awaiting requested review from malb

Assignees

@mkoeppe mkoeppe

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mkoeppe @dimpase