Skip to content

Conversation

@mamoodi
Copy link
Collaborator

@mamoodi mamoodi commented Nov 3, 2025

  • I have read and reviewed the documentation changes to the best of my ability.
  • If the change is significant, I have run the documentation site locally and confirmed it renders as expected.

Summary of changes

Disable auto sync for sdk on branches. This workflow makes it so the PRs have commits that the author didn't write. Disabling on branches.

Copy link
Collaborator

@enyst enyst left a comment

Choose a reason for hiding this comment

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

Yes, it’s not just you who finds it problematic, I think Robert raised also a concern that reverts become hard because of unrelated changes merged in; I do think we need to change how we do this.

FWIW I had a branch somewhere where the agent was trying to enable this on weekly, and make it create a PR, instead of automatic, not sure where it is… 😂

@mamoodi mamoodi merged commit 5053ca6 into main Nov 3, 2025
2 checks passed
@mamoodi mamoodi deleted the mh/disable-sync-branch branch November 3, 2025 15:30
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