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

Update golang/content.md to explain when and how to install git-lfs #2389

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

danielorbach
Copy link
Contributor

The Go toolchain uses the local git installation to fetch archives of modules from Git servers directly. My company commits large files using Git LFS which happen to be part of our Go module source tree. Without git-lfs installed, the local git installation does not automatically resolve those files which causes "checksum mismatch" against the committed go.sum.

This pull-request aims to ease the strain on developers that encounter that unfortunately under-documented scenario, and lower the barrier of entry for companies to the Golang world.

The Go toolchain uses the local git installation to fetch archives of modules from Git servers directly. My company commits large files using Git LFS which happen to be part of our Go module source tree. Without _git-lfs_ installed, the local git installation does not automatically resolve those files which causes "_checksum mismatch_" against the committed `go.sum`.
@danielorbach
Copy link
Contributor Author

@jbleduigou I saw you were the last to commit on this file, can you guide me towards improving this pull-request?

This is my first time contributing to this repository, I will be glad to make any adjustments necessary to fit the existing conventions and quality requirements.

@danielorbach
Copy link
Contributor Author

This pull-request follows my original suggestion to update the base golang image at docker-library/golang#492

golang/content.md Outdated Show resolved Hide resolved
Accept the more succinct version suggested in the pull-request.

Co-authored-by: Tianon Gravi <admwiggin@gmail.com>
@danielorbach danielorbach requested a review from tianon December 12, 2023 18:18
@tianon tianon merged commit 44d2bd6 into docker-library:master Dec 12, 2023
5 checks passed
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