Skip to content

Conversation

SilasPeters
Copy link
Member

No description provided.

@SilasPeters SilasPeters self-assigned this May 3, 2025
@SilasPeters SilasPeters requested a review from SamStaijen May 3, 2025 18:30
@SilasPeters SilasPeters requested review from SamStaijen and removed request for SamStaijen May 16, 2025 08:30
@SilasPeters
Copy link
Member Author

@SamStaijen I added the explanation of the workflow

@stickyPiston
Copy link
Member

Do we want to merge via a PR to main? This does cause main to "fall behind and be ahead" of development, since the commit hashes will be different.

@SilasPeters
Copy link
Member Author

Do we want to merge via a PR to main? This does cause main to "fall behind and be ahead" of development, since the commit hashes will be different.

This workflow does not recreate the commits afaik, and I will set a branch rule so that dev-to-main-* branches can only be merged through a PR with rebasing. Thus, the main branch will have the same hashes as development

@stickyPiston
Copy link
Member

Be careful though: rebasing through github will still alter the commit hashes, and cause the fall behind and be ahead behaviour. Unfortunately there is no way in github to do this without causing this behaviour.

@SilasPeters
Copy link
Member Author

Then I shall test this, because we did the exact same at https://github.com/team-zomsa/aplib.net, and that worked like I intend to now

@stickyPiston
Copy link
Member

Ah, Olaf and I tried to do something similar in static-sticky, but we didn't succeed, so if you manage to get it to work, we can port to other repos as well.

@SilasPeters
Copy link
Member Author

Ah, Olaf and I tried to do something similar in static-sticky, but we didn't succeed, so if you manage to get it to work, we can port to other repos as well.

Bruh @stickyPiston you are right... Let me ask someone who might know more. My guess is that we should rebase through a CLI. Perhaps we can let another workflow rebase with the CLI on approval automatically.

@SilasPeters SilasPeters marked this pull request as draft June 22, 2025 16:55
@SilasPeters SilasPeters closed this Sep 6, 2025
@SilasPeters SilasPeters deleted the chore/pr-for-dev-to-main branch September 6, 2025 16:40
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