Skip to content

Delete .cirrus.yml - #123

Merged
dpo merged 2 commits into
mainfrom
remove-cirrus-yml-20260917
Sep 18, 2026
Merged

dpo merged 2 commits into
mainfrom
remove-cirrus-yml-20260917

Conversation

@dpo

@dpo dpo commented Sep 17, 2026

Copy link
Copy Markdown
Member

Remove the obsolete Cirrus CI configuration.

Copilot AI lite review requested due to automatic review settings September 17, 2026 20:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔵 Needs a closer look

Update the stale Cirrus badge, link, and FreeBSD coverage documentation.

Pull request overview

This pull request removes the obsolete Cirrus CI configuration.

Changes:

  • Deletes .cirrus.yml.
  • README CI badge and FreeBSD coverage references still require updating.
File summaries
File Description
.cirrus.yml Removes legacy Cirrus CI setup.
Review details

Suppressed comments (2)

.cirrus.yml:1

  • Please remove or replace the Cirrus badge in README.md as part of this deletion. README.md:7 still links to cirrus-ci.com/github/JuliaSmoothOptimizers/BenchmarkProfiles.jl, so after removing this configuration the repository will advertise a stale CI provider and a non-updating build status.
    .cirrus.yml:1
  • Deleting the only Cirrus configuration leaves the Cirrus status badge and link in README.md pointing at a CI service that will no longer run; the table also still advertises FreeBSD coverage that this PR removes. Please remove the Cirrus badge and update the platform label in the same change so the documented CI status remains accurate.
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

github-actions Bot commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor
Package name latest stable
SolverBenchmark

@dpo
dpo merged commit 5fca54d into main Sep 18, 2026
4 of 14 checks passed
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