Skip to content

Update CI to use setup-gap@v3; require curlInterface 2.4.0 for timeout support#98

Merged
cdwensley merged 2 commits into
masterfrom
mh/ci
Jul 21, 2026
Merged

Update CI to use setup-gap@v3; require curlInterface 2.4.0 for timeout support#98
cdwensley merged 2 commits into
masterfrom
mh/ci

Conversation

@fingolfin

@fingolfin fingolfin commented Jul 17, 2026

Copy link
Copy Markdown
Member

This reverts commit 9a1dabb / relands PR #95.

Moreover, it changes utils to require curlInterface 2.4.0, as that is needed for timeout support in Download.

@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.24%. Comparing base (0486034) to head (419d7b8).

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
+ Coverage   85.56%   86.24%   +0.68%     
==========================================
  Files          32       32              
  Lines        1905     1905              
==========================================
+ Hits         1630     1643      +13     
+ Misses        275      262      -13     

see 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cdwensley

Copy link
Copy Markdown
Collaborator

Currently PackageInfo.g has GAP >= 4.12 and the minimal test is failing.
Try 4.13 and see what happens?

@fingolfin

Copy link
Copy Markdown
Member Author

No, I think the problem is flaky tests. I am looking into a more permanent solution right now...

@fingolfin
fingolfin marked this pull request as ready for review July 18, 2026 23:31
Install that version explicitly so the minimal GAP job does not use
the older bundled curlInterface, which silently ignores maxTime.

Co-authored-by: Codex <codex@openai.com>
@fingolfin fingolfin changed the title Reapply "Update CI to use setup-gap@v3" Update CI to use setup-gap@v3; require curlInterface 2.4.0 for timeout support Jul 19, 2026
@fingolfin

Copy link
Copy Markdown
Member Author

@cdwensley this should be ready now :-)

@fingolfin
fingolfin requested a review from cdwensley July 21, 2026 10:00
@cdwensley
cdwensley merged commit 2cc2688 into master Jul 21, 2026
5 checks passed
@cdwensley
cdwensley deleted the mh/ci branch July 21, 2026 12:04
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