Skip to content

Dolt Stash CLI Migration#9369

Merged
NathanGabrielson merged 11 commits intomainfrom
nathan/stash2
Jun 24, 2025
Merged

Dolt Stash CLI Migration#9369
NathanGabrielson merged 11 commits intomainfrom
nathan/stash2

Conversation

@NathanGabrielson
Copy link
Copy Markdown
Contributor

@NathanGabrielson NathanGabrielson commented Jun 18, 2025

Part 2 of #7623

Migrates dolt stash cli command to use dolt_stash stored procedure

@coffeegoddd
Copy link
Copy Markdown
Contributor

@NathanGabrielson DOLT

comparing_percentages
100.000000 to 100.000000
version result total
ff2cb0d ok 5937457
version total_tests
ff2cb0d 5937457
correctness_percentage
100.0

@coffeegoddd
Copy link
Copy Markdown
Contributor

@coffeegoddd DOLT

comparing_percentages
100.000000 to 100.000000
version result total
f1a8441 ok 5937457
version total_tests
f1a8441 5937457
correctness_percentage
100.0

Copy link
Copy Markdown
Contributor

@macneale4 macneale4 left a comment

Choose a reason for hiding this comment

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

the dEnv comment is going to force you to rewrite a bunch, so I'm going to stop there.

@coffeegoddd
Copy link
Copy Markdown
Contributor

@NathanGabrielson DOLT

comparing_percentages
100.000000 to 100.000000
version result total
9cb03ae ok 5937457
version total_tests
9cb03ae 5937457
correctness_percentage
100.0

@coffeegoddd
Copy link
Copy Markdown
Contributor

@coffeegoddd DOLT

comparing_percentages
100.000000 to 100.000000
version result total
9c46ad0 ok 5937457
version total_tests
9c46ad0 5937457
correctness_percentage
100.0

@coffeegoddd
Copy link
Copy Markdown
Contributor

@NathanGabrielson DOLT

comparing_percentages
100.000000 to 100.000000
version result total
4246cae ok 5937457
version total_tests
4246cae 5937457
correctness_percentage
100.0

@coffeegoddd
Copy link
Copy Markdown
Contributor

@coffeegoddd DOLT

comparing_percentages
100.000000 to 100.000000
version result total
afb2862 ok 5937457
version total_tests
afb2862 5937457
correctness_percentage
100.0

@NathanGabrielson NathanGabrielson marked this pull request as ready for review June 23, 2025 16:17
@coffeegoddd
Copy link
Copy Markdown
Contributor

@NathanGabrielson DOLT

comparing_percentages
100.000000 to 100.000000
version result total
f0b8339 ok 5937457
version total_tests
f0b8339 5937457
correctness_percentage
100.0

@coffeegoddd
Copy link
Copy Markdown
Contributor

@NathanGabrielson DOLT

comparing_percentages
100.000000 to 100.000000
version result total
1f08c69 ok 5937457
version total_tests
1f08c69 5937457
correctness_percentage
100.0

Copy link
Copy Markdown
Contributor

@macneale4 macneale4 left a comment

Choose a reason for hiding this comment

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

Getting close!!

@NathanGabrielson NathanGabrielson changed the title Rough draft for stash migration Dolt Stash CLI Migration Jun 23, 2025
Copy link
Copy Markdown
Contributor

@macneale4 macneale4 left a comment

Choose a reason for hiding this comment

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

LGTM! Ship when tests all pass!

                                \
                                  \   O,
                        \___________\/ )_________/
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ \~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                        \

@coffeegoddd
Copy link
Copy Markdown
Contributor

@NathanGabrielson DOLT

comparing_percentages
100.000000 to 100.000000
version result total
18dc62e ok 5937457
version total_tests
18dc62e 5937457
correctness_percentage
100.0

@NathanGabrielson NathanGabrielson merged commit 6e0b8e9 into main Jun 24, 2025
26 of 27 checks passed
@Hydrocharged Hydrocharged deleted the nathan/stash2 branch December 15, 2025 06:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants