Skip to content

Adding new copy-tags command#9085

Merged
fulghum merged 2 commits intomainfrom
fulghum/copy-tags
Apr 10, 2025
Merged

Adding new copy-tags command#9085
fulghum merged 2 commits intomainfrom
fulghum/copy-tags

Conversation

@fulghum
Copy link
Copy Markdown
Contributor

@fulghum fulghum commented Apr 8, 2025

No description provided.

@fulghum fulghum force-pushed the fulghum/copy-tags branch 2 times, most recently from 000a264 to 3c4753e Compare April 8, 2025 22:20
@fulghum fulghum marked this pull request as ready for review April 8, 2025 22:22
@fulghum fulghum force-pushed the fulghum/copy-tags branch from 3c4753e to a5dc3a6 Compare April 8, 2025 22:27
@coffeegoddd
Copy link
Copy Markdown
Contributor

@fulghum DOLT

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

@dolthub dolthub deleted a comment from coffeegoddd Apr 8, 2025
@dolthub dolthub deleted a comment from coffeegoddd Apr 8, 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.

Several comments. Nothing that merits another review though.

@coffeegoddd
Copy link
Copy Markdown
Contributor

@fulghum DOLT

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

@fulghum fulghum force-pushed the fulghum/copy-tags branch from dbe9537 to a7dcd7d Compare April 10, 2025 18:26
@coffeegoddd
Copy link
Copy Markdown
Contributor

@fulghum DOLT

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

@fulghum fulghum merged commit cb28d95 into main Apr 10, 2025
21 checks passed
@fulghum fulghum deleted the fulghum/copy-tags branch April 10, 2025 20:23
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