Skip to content
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

Parse renamed files #10

Merged
merged 1 commit into from
Jan 2, 2018
Merged

Parse renamed files #10

merged 1 commit into from
Jan 2, 2018

Conversation

dahlia
Copy link
Contributor

@dahlia dahlia commented Nov 3, 2017

This patch adds Renamed constructor to FileStatus type and modifies fileStatus function so that it can parse patterns such as the following diff:

diff --git a/foo.txt b/bar.txt
similarity index 100%
rename from foo.txt
rename to bar.txt

dahlia added a commit to dahlia/checkmate that referenced this pull request Nov 3, 2017
dahlia added a commit to dahlia/checkmate that referenced this pull request Nov 3, 2017
dahlia added a commit to dahlia/checkmate that referenced this pull request Nov 3, 2017
@dahlia
Copy link
Contributor Author

dahlia commented Nov 4, 2017

The build was failed during installing hspec-discover library. Seems it uses too old version of hspec. Wouldn't it be better to use more recent version of hspec?

Added: Oh, it already was reported as an issue #9.

@dahlia dahlia force-pushed the parse-renamed-file branch from cb86ec8 to f43da00 Compare January 2, 2018 16:40
@dahlia
Copy link
Contributor Author

dahlia commented Jan 2, 2018

Rebased the patch on the latest master, and the CI build succeeded as well! @mulby could you review this PR too?

@mulby
Copy link
Owner

mulby commented Jan 2, 2018

Looks great, thanks for the contribution!

@mulby mulby merged commit a2218af into mulby:master Jan 2, 2018
@dahlia
Copy link
Contributor Author

dahlia commented Jan 2, 2018

Thanks for merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants