Skip to content

ci(illumos): clippy instead of test - #687

Merged
George-Miao merged 5 commits into
compio-rs:masterfrom
Berrysoft:ci-ignore-illumos
Feb 14, 2026
Merged

George-Miao merged 5 commits into
compio-rs:masterfrom
Berrysoft:ci-ignore-illumos

Conversation

@Berrysoft

@Berrysoft Berrysoft commented Feb 14, 2026 •

Copy link
Copy Markdown
Member

Closes #682

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR modifies the Illumos CI test workflow to ignore test failures by adding continue-on-error: true to the test step. This means the workflow will complete successfully even if the actual tests fail.

Changes:

  • Added continue-on-error: true to the Illumos test step in the CI workflow

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci_test_illumos.yml Outdated
Comment thread .github/workflows/ci_test_illumos.yml
@George-Miao George-Miao added bug Something isn't working ci github action and other ci-related change labels Feb 14, 2026
@Berrysoft Berrysoft changed the title ci: ignore errors of Illumos tests ci(illumos): clippy instead of test Feb 14, 2026

@George-Miao George-Miao left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@George-Miao
George-Miao merged commit da5ea28 into compio-rs:master Feb 14, 2026
2 checks passed
@Berrysoft
Berrysoft deleted the ci-ignore-illumos branch February 14, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working ci github action and other ci-related change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants