Skip to content

Comments

ci: Remove unused strategy sections from non-matrix jobs#566

Open
kevinjqliu wants to merge 1 commit intoapache:mainfrom
kevinjqliu:kevinjqliu/fix-github-workflow-syntax
Open

ci: Remove unused strategy sections from non-matrix jobs#566
kevinjqliu wants to merge 1 commit intoapache:mainfrom
kevinjqliu:kevinjqliu/fix-github-workflow-syntax

Conversation

@kevinjqliu
Copy link
Contributor

Summary

Removes unnecessary strategy sections from jobs that don't use a matrix.

Changes

  • Removed strategy: fail-fast: false from ubuntu, macos, windows jobs in test.yml
  • Removed strategy: fail-fast: false from sanitizer-test job in sanitizer_test.yml

Rationale

The fail-fast option only applies to matrix jobs where multiple configurations run in parallel. For single jobs without a matrix, this setting has no effect and generates schema warnings.

Jobs with matrices (like meson and verify) retain their strategy configuration.

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.

3 participants