Skip to content

ci: ignore unknown features#3

Merged
rymnc merged 1 commit into
mainfrom
ci/hack-ignore-unknown-features
Jul 21, 2026
Merged

ci: ignore unknown features#3
rymnc merged 1 commit into
mainfrom
ci/hack-ignore-unknown-features

Conversation

@rymnc

@rymnc rymnc commented Jul 21, 2026

Copy link
Copy Markdown
Member

This pull request updates the CI workflow configuration to improve compatibility when running checks with multiple feature combinations. The main change is the addition of the --ignore-unknown-features flag to both the clippy and nextest steps, which helps prevent failures caused by unknown features in the workspace.

CI workflow improvements:

  • Added --ignore-unknown-features to the cargo hack clippy step in .github/workflows/ci.yml to avoid errors from unknown features when linting.
  • Added --ignore-unknown-features to the cargo hack nextest run step in .github/workflows/ci.yml to ensure tests don't fail due to unknown features.

@rymnc rymnc self-assigned this Jul 21, 2026
Copilot AI review requested due to automatic review settings July 21, 2026 10:07
@rymnc
rymnc requested a review from oskarth as a code owner July 21, 2026 10:07

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.

Pull request overview

Updates the GitHub Actions CI workflow to make cargo-hack feature-powerset runs resilient when some crates don’t define all features by ignoring unknown features, reducing avoidable CI failures while keeping broad feature coverage.

Changes:

  • Add --ignore-unknown-features to cargo hack clippy --feature-powerset invocation.
  • Add --ignore-unknown-features to cargo hack nextest run --feature-powerset invocation.

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

@rymnc
rymnc merged commit 666bbcd into main Jul 21, 2026
1 check passed
@rymnc
rymnc deleted the ci/hack-ignore-unknown-features branch July 21, 2026 10:09
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