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

optional git-lfs #1171

Merged
merged 1 commit into from
Dec 10, 2023
Merged

optional git-lfs #1171

merged 1 commit into from
Dec 10, 2023

Conversation

Byron
Copy link
Member

@Byron Byron commented Dec 10, 2023

Maybe this also means that one should prefer to not use git-lfs at all
and keep data in the packs themselves, it still shouldn't be too much overall.

Butl... for now, let's try to keep it to make the experience.

@Byron Byron force-pushed the lfs-independence branch 4 times, most recently from 2b4d04d to 762cc39 Compare December 10, 2023 17:56
… where possible

Maybe this also means that one should prefer to not use git-lfs at all
and keep data in the packs themselves, it still shouldn't be too much overall.

Butl... for now, let's try to keep it to make the experience.

Here we choose the quick way out by simply checking in the archives in question
and by disabling git-lfs for them.

That way, most of the scripts still run, but we don't have to square the circle
trying to get these scripts portable, or spend a lot of time figuring out why
some baselines don't match perfectly (on windows).
@Byron Byron merged commit b6f2b81 into main Dec 10, 2023
19 checks passed
@Byron Byron deleted the lfs-independence branch December 10, 2023 19:20
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.

1 participant