Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Supports "sparo checkout -" #75

Merged
merged 2 commits into from
May 31, 2024
Merged

Supports "sparo checkout -" #75

merged 2 commits into from
May 31, 2024

Conversation

chengcyber
Copy link
Member

@chengcyber chengcyber commented May 30, 2024

Basic Checks

Have you run rush change for this change?

  • Yes
  • No

If No, please run rush change before, this is necessary.

If adding a new feature, the PR's description includes:

  • Reason for adding this feature
  • How to use
  • A basic example

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

If yes, please describe the impact and migration path for existing applications:

Summary

Previously, "sparo checkout" throws an error when parsing "-" branch name. Now "sparo checkoout -" can work similiar to "git checkout -" does

Detail

When parsing "-" branch, sparo reads the logs from .git/logs/HEAD from and get the previous branch name. With this branch name, do the following logic as normal.

How to test it

Local

@chengcyber chengcyber force-pushed the feat-checkout-dash branch 2 times, most recently from d316878 to 91e5de3 Compare May 30, 2024 05:37
@chengcyber chengcyber force-pushed the feat-checkout-dash branch from 91e5de3 to 7ea9bdc Compare May 30, 2024 16:06
@chengcyber chengcyber requested a review from jzhang026 May 30, 2024 16:06
@jzhang026 jzhang026 enabled auto-merge May 30, 2024 16:24
@chengcyber chengcyber disabled auto-merge May 31, 2024 18:28
@chengcyber chengcyber merged commit a045b7f into main May 31, 2024
3 checks passed
@chengcyber chengcyber deleted the feat-checkout-dash branch May 31, 2024 18:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants