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.

Sign up for GitHub

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

Jump to bottom

Git status shortcuts broken edge case: moved file #267

Open

LeZuse opened this issue Sep 3, 2018 · 1 comment

LeZuse commented Sep 3, 2018 •

edited

Loading

To reproduce simply move a checked in file from a previous commit and stage it. Git detects a moved file:

$ git status
# On branch: master  |  [*] => $e
#
➤ Changes to be committed
#
#        renamed: [1] "app.js -> app2.js"

$e1 now contains invalid filename:

$ echo $e1
/Users/zuse/Projects/pb-frontend/apps/main/"app.js -> app2.js"

$ git --version
git version 2.10.2
$ cd ~/.scm_breeze/
$ git show --oneline
8a326c2 Merge pull request #262 from ghthor/refactor_of_git_status

The text was updated successfully, but these errors were encountered:

LeZuse mentioned this issue

Git status shortcuts broken edge case: filename with spaces #268

Open

ghthor mentioned this issue

Revert to legacy status shortcuts ruby script #269

Merged

Member

ghthor commented Sep 5, 2018

I fixing this by reverting for now. We need better test coverage to handle a major change like the refactorization that was merged in #262

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment