Skip to content

WIP: Push multiple branches in the stack atomically: it's faster, plus possibly improves CODEOWNERS processing by GitHub #26

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

dimikot
Copy link
Owner

@dimikot dimikot commented Feb 23, 2025

Summary

Git supports "atomic push" of multiple branches. We now use it in git-grok:

  1. It is faster than pushing one by one.
  2. It is possible that it improves CODEOWNERS-based reviewers addition when a large stack is reordered or rebased.

How was this tested?

git grok --debug-force-push-branches

CleanShot 2025-02-23 at 01 04 57@2x

CleanShot 2025-02-23 at 01 05 41@2x

PRs in the Stack

(The stack is managed by git-grok.)

dimikot added a commit that referenced this pull request Feb 23, 2025
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
dimikot added a commit that referenced this pull request Feb 23, 2025
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
@dimikot dimikot force-pushed the grok/dimikot/push-multiple-branches-in-the-stack-atomically-it-s-faster-plus-possible-improves-codeowners-processing-by-github-to-main-0608 branch from 0608b24 to e40068b Compare February 23, 2025 09:25
@dimikot dimikot changed the title Push multiple branches in the stack atomically: it's faster, plus possible improves CODEOWNERS processing by GitHub Push multiple branches in the stack atomically: it's faster, plus possibly improves CODEOWNERS processing by GitHub Feb 23, 2025
dimikot added a commit that referenced this pull request Feb 24, 2025
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
@dimikot dimikot force-pushed the grok/dimikot/push-multiple-branches-in-the-stack-atomically-it-s-faster-plus-possible-improves-codeowners-processing-by-github-to-main-0608 branch from e40068b to 515bfe7 Compare February 24, 2025 06:47
dimikot added a commit that referenced this pull request Feb 24, 2025
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
@dimikot dimikot force-pushed the grok/dimikot/push-multiple-branches-in-the-stack-atomically-it-s-faster-plus-possible-improves-codeowners-processing-by-github-to-main-0608 branch from 515bfe7 to 7b697d1 Compare February 24, 2025 06:48
dimikot added a commit that referenced this pull request Feb 24, 2025
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
@dimikot dimikot force-pushed the grok/dimikot/push-multiple-branches-in-the-stack-atomically-it-s-faster-plus-possible-improves-codeowners-processing-by-github-to-main-0608 branch from 7b697d1 to b4c83ff Compare February 24, 2025 06:51
dimikot added a commit that referenced this pull request Feb 24, 2025
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
@dimikot dimikot force-pushed the grok/dimikot/push-multiple-branches-in-the-stack-atomically-it-s-faster-plus-possible-improves-codeowners-processing-by-github-to-main-0608 branch from b4c83ff to 727612d Compare February 24, 2025 06:53
dimikot added a commit that referenced this pull request Feb 24, 2025
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
@dimikot dimikot force-pushed the grok/dimikot/push-multiple-branches-in-the-stack-atomically-it-s-faster-plus-possible-improves-codeowners-processing-by-github-to-main-0608 branch from 727612d to 8b0e752 Compare February 24, 2025 08:22
@dimikot dimikot mentioned this pull request Feb 24, 2025
@dimikot dimikot changed the base branch from main to grok/dimikot/fix-test-for-python-3-8-to-main-b8f1 February 24, 2025 08:22
dimikot added a commit that referenced this pull request Feb 24, 2025
## Summary

TypeError: 'type' object is not subscriptable

## How was this tested?

CI

## PRs in the Stack
- #26
- ➡ #27

(The stack is managed by
[git-grok](https://github.com/dimikot/git-grok).)
Base automatically changed from grok/dimikot/fix-test-for-python-3-8-to-main-b8f1 to main February 24, 2025 08:29
dimikot added a commit that referenced this pull request Feb 24, 2025
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
@dimikot dimikot force-pushed the grok/dimikot/push-multiple-branches-in-the-stack-atomically-it-s-faster-plus-possible-improves-codeowners-processing-by-github-to-main-0608 branch from 8b0e752 to 0ae73db Compare February 24, 2025 08:29
…sible improves CODEOWNERS processing by GitHub

Pull Request: #26 (main)
@dimikot dimikot force-pushed the grok/dimikot/push-multiple-branches-in-the-stack-atomically-it-s-faster-plus-possible-improves-codeowners-processing-by-github-to-main-0608 branch from 0ae73db to 0a1e258 Compare February 25, 2025 06:36
@dimikot dimikot changed the title Push multiple branches in the stack atomically: it's faster, plus possibly improves CODEOWNERS processing by GitHub WIP: Push multiple branches in the stack atomically: it's faster, plus possibly improves CODEOWNERS processing by GitHub Apr 13, 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.

1 participant