Skip to content

diff: capture both old and new of index and mode #73

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

Merged
merged 4 commits into from
Mar 26, 2022

Conversation

aymanbagabas
Copy link
Contributor

@aymanbagabas aymanbagabas commented Mar 23, 2022

This adds some missing information needed to constructe a full-index
diff from DiffFile struct type. It adds old/new file mode, index, and
name.

  • Add old/new mode test case

Copy link
Member

@unknwon unknwon left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Have few comments and please add corresponding unit tests.

@aymanbagabas aymanbagabas force-pushed the diff-file branch 2 times, most recently from bcec7eb to d16f88e Compare March 24, 2022 18:23
@aymanbagabas aymanbagabas requested a review from unknwon March 24, 2022 19:25
@unknwon
Copy link
Member

unknwon commented Mar 25, 2022

GitHub Actions did not pick up, I'll close and re-open to trigger it again.

@unknwon unknwon closed this Mar 25, 2022
@unknwon unknwon reopened this Mar 25, 2022
@unknwon
Copy link
Member

unknwon commented Mar 25, 2022

Thanks for the followup! Overall LGTM, asked few remaining questions :)

This adds some missing information needed to constructe a full-index
diff from DiffFile struct type. It adds old/new file mode, index, and
name.
@unknwon unknwon changed the title Add diff files info to DiffFile diff: capture both old and new of index and mode Mar 26, 2022
@unknwon unknwon merged commit 2a8fce5 into gogs:master Mar 26, 2022
@unknwon
Copy link
Member

unknwon commented Mar 26, 2022

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