Skip to content

fix(ci): ignore Make scheduling barriers in target checks - #1032

Merged
CaliLuke merged 1 commit into
mainfrom
luca/ci-make-wait
Oct 21, 2026
Merged

CaliLuke merged 1 commit into
mainfrom
luca/ci-make-wait

Conversation

@CaliLuke

Copy link
Copy Markdown
Owner

GNU Make reports .WAIT scheduling barriers between prerequisites of .NOTPARALLEL targets. The CI-contract parser now excludes those markers when comparing build targets, while preserving prerequisite order and all existing gate requirements.

Regression checks cover leading, repeated, trailing, and barrier-only lists and confirm that missing prerequisites and targets remain errors. The captured CI format reproduces the failure before the fix and passes after it; native contract lint and shell syntax checks also pass. Runtime code and Make targets are unchanged.

@CaliLuke
CaliLuke merged commit 2c22aa8 into main Oct 21, 2026
0 of 16 checks passed
@coderabbitai

coderabbitai Bot commented Oct 21, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: c5831d18-fc47-4f1c-98d4-8bda46d4b41d
📥 Commits

Reviewing files that changed from the base of the PR and between 75851c4 and 1644ea5.

📒 Files selected for processing: 1
  • scripts/lint_ci_contract.sh
+-------------------------------------------------+
| Artificial intelligence, natural bug detection. |
+-------------------------------------------------+
  \
   \     /)/)
        ( '')
        /  |
      o(___)
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@CaliLuke
CaliLuke deleted the luca/ci-make-wait branch October 22, 2026 20:19
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.

1 participant