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

ReleaseNotes: duplicated files are hard links #229

Merged
merged 1 commit into from
Feb 20, 2019

Conversation

PhilipOakley
Copy link
Contributor

Inform users that Windows Explorer double counts file sizes for
hard links.

fixes: #1977 (git-for-windows/git#1997)

Signed-off-by: Philip Oakley philipoakley@iee.org

I haven't tested this with a full build and install yet, but it looked simple enough, apart from a minor concern about the '#' in the updated markdown text.

A friendly reminder for new contributors:

What is DCO sign-off? Why is it required?

DCO stands for Developer's Certificate of Origin.

We require all commits to be DCO signed-off in case we need to track down and handle legal and/or technical issues.

To DCO sign-off your commits, you could run the git-commit command with the -s option or just add a line in your commit message in this format:

        Signed-off-by: yourFirstName yourLastName <yourEmail@example.org>

For more information, check out how PRs are checked for DCO sign-off: https://github.com/probot/dco#how-it-works .

Thank you very much.

Inform users that Windows Explorer double counts file sizes for
hard links.

fixes: #1977 (git-for-windows/git#1997)

Signed-off-by: Philip Oakley <philipoakley@iee.org>
@dscho dscho merged commit b43b52c into git-for-windows:master Feb 20, 2019
@dscho
Copy link
Member

dscho commented Feb 20, 2019

Thanks!

@PhilipOakley
Copy link
Contributor Author

Yay, the release notes formatted fine on the brand new

$ git --version
git version 2.21.0.rc2.windows.1

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