Skip to content

Fix the space insufficent problem in github ci#41576

Merged
vbraun merged 11 commits intosagemath:developfrom
cxzhong:patch-1
Feb 11, 2026
Merged

Fix the space insufficent problem in github ci#41576
vbraun merged 11 commits intosagemath:developfrom
cxzhong:patch-1

Conversation

@cxzhong
Copy link
Copy Markdown
Contributor

@cxzhong cxzhong commented Feb 2, 2026

Fix #41575
and we test on macos-15 and 26, we do not test macos-14. and in this ci test, we only test build job.

📝 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

@cxzhong cxzhong marked this pull request as draft February 2, 2026 08:41
@cxzhong cxzhong changed the title Test build CI Solve the space insufficent problem Feb 2, 2026
@cxzhong cxzhong requested review from tobiasdiez and vbraun February 2, 2026 09:43
@cxzhong cxzhong marked this pull request as ready for review February 2, 2026 09:43
@cxzhong cxzhong added github actions Pull requests that update GitHub Actions code p: blocker / 1 s: needs review and removed s: needs review labels Feb 2, 2026
@cxzhong cxzhong requested a review from kwankyu February 2, 2026 09:48
Removed 'centos-stream-9-python3.12' from the list of platforms.
Updated CI workflow to include version 26 and allow tests to continue on error.
@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 2, 2026

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

@cxzhong cxzhong requested a review from dimpase February 2, 2026 10:54
Updated the action used to maximize build disk space and modified its configuration options.
@cxzhong cxzhong changed the title Solve the space insufficent problem Fix the space insufficent problem in github ci Feb 2, 2026
Copy link
Copy Markdown
Collaborator

@kwankyu kwankyu left a comment

Choose a reason for hiding this comment

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

LGTM.

@fchapoton fchapoton added the p: CI fix merged before running CI tests label Feb 4, 2026
vbraun pushed a commit to vbraun/sage that referenced this pull request Feb 5, 2026
sagemathgh-41576: Fix the space insufficent problem in github ci
    
<!-- ^ Please provide a concise and informative title. -->
<!-- ^ Don't put issue numbers in the title, do this in the PR
description below. -->
<!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method
to calculate 1 + 2". -->
<!-- v Describe your changes below in detail. -->
<!-- v Why is this change required? What problem does it solve? -->
<!-- v If this PR resolves an open issue, please link to it here. For
example, "Fixes sagemath#12345". -->
Fix sagemath#41575
and we test on macos-15 and 26, we do not test macos-14. and in this ci
test, we only test build job.

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [ ] 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

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#41576
Reported by: Chenxin Zhong
Reviewer(s): Kwankyu Lee
vbraun pushed a commit to vbraun/sage that referenced this pull request Feb 7, 2026
sagemathgh-41576: Fix the space insufficent problem in github ci
    
<!-- ^ Please provide a concise and informative title. -->
<!-- ^ Don't put issue numbers in the title, do this in the PR
description below. -->
<!-- ^ For example, instead of "Fixes sagemath#12345" use "Introduce new method
to calculate 1 + 2". -->
<!-- v Describe your changes below in detail. -->
<!-- v Why is this change required? What problem does it solve? -->
<!-- v If this PR resolves an open issue, please link to it here. For
example, "Fixes sagemath#12345". -->
Fix sagemath#41575
and we test on macos-15 and 26, we do not test macos-14. and in this ci
test, we only test build job.

### 📝 Checklist

<!-- Put an `x` in all the boxes that apply. -->

- [ ] 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

<!-- List all open PRs that this PR logically depends on. For example,
-->
<!-- - sagemath#12345: short description why this is a dependency -->
<!-- - sagemath#34567: ... -->
    
URL: sagemath#41576
Reported by: Chenxin Zhong
Reviewer(s): Kwankyu Lee
@vbraun vbraun merged commit 7061f7f into sagemath:develop Feb 11, 2026
34 of 36 checks passed
@cxzhong cxzhong deleted the patch-1 branch February 13, 2026 03:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github actions Pull requests that update GitHub Actions code p: blocker / 1 p: CI fix merged before running CI tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Build CI fails

4 participants