Skip to content

Conversation

@spiral-ladder
Copy link
Contributor

@spiral-ladder spiral-ladder commented Oct 30, 2025

Ensure bindings are committed with a check in the CI.

This should help us avoid situations like #27

@spiral-ladder spiral-ladder self-assigned this Oct 30, 2025
@spiral-ladder spiral-ladder force-pushed the bing/ci-bindings branch 4 times, most recently from dbe5598 to 441f122 Compare October 30, 2025 15:28
@spiral-ladder
Copy link
Contributor Author

spiral-ladder commented Oct 30, 2025

See failure in 441f122 to see what it looks like if it fails (also fixed the command in the latest commit)

@spiral-ladder spiral-ladder marked this pull request as ready for review October 30, 2025 15:31
@spiral-ladder spiral-ladder requested a review from a team as a code owner October 30, 2025 15:31
This commit deliberately checks for no diffs to show failure.
@spiral-ladder spiral-ladder force-pushed the bing/ci-bindings branch 5 times, most recently from 6ed8e50 to 4fd2b4e Compare October 30, 2025 16:04
Copy link
Member

@nflaig nflaig left a comment

Choose a reason for hiding this comment

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

lgtm

@nflaig nflaig merged commit b11a5e8 into main Oct 30, 2025
2 checks passed
@nflaig nflaig deleted the bing/ci-bindings branch October 30, 2025 18:03
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