Skip to content

CI: Push wasm commit on changes only #603

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 18, 2025

Conversation

ChinYikMing
Copy link
Collaborator

When rerunning the deploy wasm CI, previously successful jobs may fail due to a reproducible error:
nothing to commit, working tree clean
Error: Process completed with exit code 1.

This occurs because the successful deployment has already committed the changes. To prevent this, the workflow should check whether there are any staged changes before attempting to commit. If there’s nothing to commit, skip the push step. Otherwise, proceed with committing and pushing the new changes.

The error can be observed in this ci.

When rerunning the deploy wasm CI, previously successful jobs may fail
due to a reproducible error:
nothing to commit, working tree clean
Error: Process completed with exit code 1.

This occurs because the successful deployment has already committed the
changes. To prevent this, the workflow should check whether there are
any staged changes before attempting to commit. If there’s nothing to
commit, skip the push step. Otherwise, proceed with committing and
pushing the new changes.
@ChinYikMing ChinYikMing requested a review from jserv July 18, 2025 16:44
Copy link

Bito Automatic Review Skipped - Files Excluded

Bito didn't auto-review this change because all changed files are in the exclusion list for automatic reviews. No action is needed if you didn't intend for the agent to review it. Otherwise, to manually trigger a review, type /review in a comment and save.
You can change the excluded files settings here, or contact your Bito workspace admin at jserv.tw@gmail.com.

@jserv jserv merged commit a09ca99 into sysprog21:master Jul 18, 2025
8 checks passed
@ChinYikMing ChinYikMing deleted the ci-wasm-commit-on-change branch July 19, 2025 03:08
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