Skip to content

Fix sync upstream workflow#561

Open
Salmaan-M wants to merge 2 commits intofossasia:mainfrom
Salmaan-M:fix-sync-upstream-workflow
Open

Fix sync upstream workflow#561
Salmaan-M wants to merge 2 commits intofossasia:mainfrom
Salmaan-M:fix-sync-upstream-workflow

Conversation

@Salmaan-M
Copy link
Copy Markdown
Contributor

📌 Fixes

N/A


📝 Summary of Changes

-Added a GitHub Actions workflow to automatically sync forks with the upstream repository
-Ensures only forks that are behind (and not ahead) are updated
-Prevents overwriting contributor changes by skipping diverged forks
-Added pagination support to handle large numbers of forks
-Included retry logic and improved logging for better reliability


📸 Screenshots / Demo (if UI-related)

Not applicable (workflow/automation change)


✅ Checklist

  • I’ve tested my changes locally
  • I’ve added tests (if applicable)
  • I’ve updated documentation (if applicable)
  • My code follows the project’s code style guidelines

👀 Reviewer Notes

This workflow is designed to safely automate syncing of forks without affecting contributors' work. Forks with diverging commits are skipped intentionally to avoid conflicts or data loss. Some forks may not sync due to GitHub permission limitations

Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Sorry @Salmaan-M, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@Salmaan-M
Copy link
Copy Markdown
Contributor Author

This is an important enhancement because it helps keep forks automatically up to date with the upstream repository, reducing manual effort for contributors. The workflow safely syncs only non-diverging forks and improves overall project maintainability.

Please review @vedansh-5 @hpdang when convenient.
Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant