Skip to content

Conversation

@Bhaal22
Copy link
Contributor

@Bhaal22 Bhaal22 commented Mar 16, 2017

No description provided.

@Bhaal22 Bhaal22 requested review from dhurum and mbrt March 16, 2017 15:41
Copy link
Contributor

@mbrt mbrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the name of the command can be improved. Maybe diff-common-files?

COMMON_ANCESTOR=$(git merge-base $LEFT_COMMIT $RIGHT_COMMIT)
FILES=$(git show --pretty="" --name-only $COMMON_ANCESTOR..HEAD)

git diff $STAT $LEFT_COMMIT $RIGHT_COMMIT $FILES
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is the risk to hit the max command line length when there are a lot of files

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.

3 participants