codeforboston / home-energy-analysis-tool

https://www.codeforboston.org/projects/
MIT License
7 stars 25 forks source link

Remove Git LFS #178

Closed AdamFinkle closed 3 weeks ago

AdamFinkle commented 1 month ago

Git Large File System is required for our repository although none of the files requires it anymore. I have searched the issues list for LFS and seen that #88 may be connected.

thadk commented 1 month ago

Thanks Adam, we opted for a different approach for #88 so we will not store the rules engine wheel inside the repository at all.

We can turn off Git LFS for the repository if someone can figure out how, and submit a PR.

140 was our previous attempt to remove it, but it wasn't fully successful.

AdamFinkle commented 1 month ago

I have found a potentially helpful guide to solving this problem. The guide's first step is to run git uninstall lfs, but the next is rewriting the history of the repository to archive the large files, and I am not sure whether this step is already done or, if not, how to do it.

I have tried taking just the first step and pushing to my branch, and would open a pull request, but GitHub says that my push is identical.

thadk commented 1 month ago

This 2mb file is saved as ifs: https://github.com/codeforboston/home-energy-analysis-tool/blob/7555497d80ca6cb574b97c5626c4d48cfa54f137/design_temp/ENERGY%20STAR%20Certified%20Homes%20County-Level%20Design%20Temperature%20Reference%20Guide.pdf#L2

AdamFinkle commented 1 month ago

This branch should have LFS turned off, but it requires rewriting every commit, requiring all PRs to be submitted before we do it. Follow this Phind exactly to replicate the steps, which you must do after submitting every PR you want to keep because these changes rewrite every commit.

thadk commented 3 weeks ago

https://github.com/codeforboston/home-energy-analysis-tool/pull/140#issuecomment-2148693591