Skip to content

test: cover all commands #18

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

Conversation

armcknight
Copy link
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
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
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.

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