Skip to content

test: cover all commands#18

Closed
armcknight wants to merge 5 commits intoBinaryBirds:armcknight/fix/checkout-param-revertedfrom
armcknight:armcknight/test/tests-covering-all-commands
Closed

test: cover all commands#18
armcknight wants to merge 5 commits intoBinaryBirds:armcknight/fix/checkout-param-revertedfrom
armcknight:armcknight/test/tests-covering-all-commands

Conversation

@armcknight
Copy link
Copy Markdown
Collaborator

@armcknight armcknight commented Aug 5, 2025

Currently set as draft as it needs the other PRs merged first to resolve build errors.

Thought this might be nice to at least have full line-of-code coverage, even if certain edge cases might not be covered (yet–that could come in the future, but not sure how much benefit would come from it)

@armcknight armcknight changed the base branch from main to armcknight/fix/checkout-param-reverted August 5, 2025 02:09
@armcknight armcknight force-pushed the armcknight/test/tests-covering-all-commands branch from 881cdcc to a26d6f4 Compare August 5, 2025 02:12
@armcknight armcknight marked this pull request as draft August 5, 2025 02:20
@tib tib deleted the branch BinaryBirds:armcknight/fix/checkout-param-reverted August 5, 2025 12:47
@tib tib closed this Aug 5, 2025
@tib
Copy link
Copy Markdown
Member

tib commented Aug 5, 2025

I accidentally clicked the wrong button and now I can't restore the branch from the UI. 🤦

Could you please push it again and re-open the PR?
Sorry about that... I’ll review it as soon as you re-open it. 🙏

@armcknight
Copy link
Copy Markdown
Collaborator Author

That's ok! I can't reopen it either since the base branch is gone, i'll reopen this once I get it rebased.

@armcknight armcknight deleted the armcknight/test/tests-covering-all-commands branch March 30, 2026 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants