Skip to content

Conversation

@rmainwork
Copy link
Contributor

@rmainwork rmainwork commented May 8, 2025

πŸ”— Relevant links

πŸ—’οΈ What

Add trigger to repo-sync workflow to trigger sync workflow on pushes to main.

🀷 Why

The repo sync runs automatically every 2 hours (or manually via workflow_dispatch). However, if a merge to main occurs outside that 2 hour window, it then requires a manual trigger of workflow_dispatch to sync the repos.

Add trigger to `repo-sync` workflow to trigger sync workflow on pushes
to `main`.
@github-actions
Copy link
Contributor

github-actions bot commented May 8, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal βœ… Ready (Inspect) Visit Preview Thu May 8 20:03:06 UTC 2025
Unified Docs API βœ… Ready (Inspect) Visit Preview Thu May 8 19:57:49 UTC 2025

@rmainwork rmainwork marked this pull request as ready for review May 8, 2025 14:57
@rmainwork rmainwork requested a review from a team as a code owner May 8, 2025 14:57
@rmainwork rmainwork requested a review from RubenSandwich May 8, 2025 14:57
Copy link
Contributor

@RubenSandwich RubenSandwich left a comment

Choose a reason for hiding this comment

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

πŸ‘πŸ»

@rmainwork
Copy link
Contributor Author

@RubenSandwich something just occurred to me with this - since the sync happens on commits to main - do we even need the scheduled sync every 2 hours? πŸ€”

@RubenSandwich
Copy link
Contributor

@RubenSandwich something just occurred to me with this - since the sync happens on commits to main - do we even need the scheduled sync every 2 hours? πŸ€”

Hmm good point.

@rmainwork
Copy link
Contributor Author

It might help cut down on this
image

@RubenSandwich
Copy link
Contributor

It might help cut down on this image

Yeah let's do it.

Since the repo sync is triggered on pushes to `main`, the 2 hour sync is
no longer required and can be removed as it is producing excessive
quantities of PRs
@rmainwork rmainwork requested a review from RubenSandwich May 8, 2025 19:46
Copy link
Contributor

@RubenSandwich RubenSandwich left a comment

Choose a reason for hiding this comment

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

πŸ‘πŸ»

@rmainwork rmainwork merged commit b4bba96 into main May 8, 2025
9 checks passed
@rmainwork rmainwork deleted the rm/trigger-repo-sync-on-pushes-to-main branch May 8, 2025 20:08
hashibot-web added a commit that referenced this pull request Sep 8, 2025
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