Skip to content

[Github actions] Modify the time slot of the refresh-sqlite-integration workflow - #3510

Merged
JanJakes merged 1 commit into
trunkfrom
run-refresh-sqlite-integration-on-unique-time-slot
Apr 21, 2026
Merged

[Github actions] Modify the time slot of the refresh-sqlite-integration workflow#3510
JanJakes merged 1 commit into
trunkfrom
run-refresh-sqlite-integration-on-unique-time-slot

Conversation

@mho22

@mho22 mho22 commented Apr 20, 2026

Copy link
Copy Markdown
Collaborator

Motivation for the change, related issues

The Publish NPM packages workflow failed today. And based on the stack trace : pushing the branch trunk was rejected because the remote is ahead. The only file modification on trunk is packages/playground/wordpress-builds/src/sqlite-database-integration/sqlite-database-integration-trunk.zip because of this commit. Which I suspect made the process of publishing NPM packages fail.

Both workflows triggered at the same time, and the other workflow pushed that commit to trunk before the publish workflow could push its version bump commit.

Implementation details

Modified cron to be'0 10 * * *' instead of same time slot as publish-npm-packages: '0 9 * * *'

Testing Instructions

Publish NPM packages workflow completion next monday.

…t run simultaneously with the publish-npm-packages
@mho22
mho22 requested review from a team, JanJakes, bgrgicak and Copilot and removed request for bgrgicak April 20, 2026 14:24
@mho22 mho22 added [Type] Bug An existing feature does not function as intended [Feature] GitHub integration dependencies Pull requests that update a dependency file labels Apr 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adjusts the scheduled time of refresh-sqlite-integration to reduce collisions with publish-npm-packages, after a recent failure caused by concurrent pushes to trunk.

Changes:

  • Move the workflow cron from 0 9 * * * to 0 10 * * *
  • Remove an outdated schedule comment

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

Comment thread .github/workflows/refresh-sqlite-integration.yml
Comment thread .github/workflows/refresh-sqlite-integration.yml

@JanJakes JanJakes left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good catch, thanks!

@JanJakes
JanJakes merged commit c555717 into trunk Apr 21, 2026
92 of 93 checks passed
@JanJakes
JanJakes deleted the run-refresh-sqlite-integration-on-unique-time-slot branch April 21, 2026 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file [Feature] GitHub integration [Type] Bug An existing feature does not function as intended

3 participants