Skip to content
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

fix: expect only persistant commands to persist (do not fail assertion because of non-persistant ones) #15

Draft
wants to merge 6 commits into
base: feat/pull
Choose a base branch
from

Conversation

kiprasmel
Copy link
Owner

@kiprasmel kiprasmel commented May 6, 2022

root issue was noticed when doing a simple git-stacked-rebase and adding
a break inside the git-rebase-todo file -- --apply would always fail
because there'd be 1 less command than expected.

@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch 4 times, most recently from 9ba7444 to d860fbc Compare May 7, 2022 23:16
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from d860fbc to 677a26a Compare May 8, 2022 05:01
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 677a26a to 5c6f9c7 Compare May 8, 2022 07:49
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 5c6f9c7 to e84378a Compare May 8, 2022 08:44
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from e84378a to 4597b24 Compare May 8, 2022 09:01
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 4597b24 to 94efe46 Compare May 8, 2022 09:03
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 94efe46 to a6c8d78 Compare May 8, 2022 09:10
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from a6c8d78 to 9e89866 Compare May 8, 2022 09:24
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 9e89866 to fc7c8c2 Compare May 12, 2022 02:10
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from fc7c8c2 to 47fe88c Compare May 28, 2022 20:58
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 47fe88c to 6339eb8 Compare May 28, 2022 22:54
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 6339eb8 to 5605b6d Compare May 28, 2022 23:17
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 5605b6d to 22ac353 Compare May 28, 2022 23:40
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 22ac353 to 97c3444 Compare May 28, 2022 23:56
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch 2 times, most recently from 1a272f3 to 84e176a Compare June 7, 2022 02:49
…n because of non-persistant ones)

root issue was noticed when doing a simple git-stacked-rebase and adding
a `break` inside the git-rebase-todo file -- `--apply` would always fail
because there'd be 1 less command than expected.

Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
…concurrent-safe

Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
Signed-off-by: Kipras Melnikovas <kipras@kipras.org>
@kiprasmel kiprasmel force-pushed the fix/expect-only-persistant-commands-to-persist branch from 84e176a to 42f69ce Compare June 7, 2022 02:53
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