Skip to content

Migrate svg files to git lfs that were not already there#364

Merged
erinxocon merged 1 commit into
masterfrom
erin/migrate-svg-to-lfs
May 27, 2026
Merged

Migrate svg files to git lfs that were not already there#364
erinxocon merged 1 commit into
masterfrom
erin/migrate-svg-to-lfs

Conversation

@erinxocon
Copy link
Copy Markdown
Member

Fix: There were 49 files that were added by someone who did not have git-lfs installed or configured properly. Hence these files were committed directly to the repository and it's now causing issues with the reorganization i am doing. This commit was generated by running

git lfs migrate import --no-rewrite $(git lfs status | grep '\.svg' | awk '{print $1}')

…models/pang12/img/fn-f11.svg,src/models/pang12/img/fn-f12.svg,src/models/pang12/img/fn-f2.svg,src/models/pang12/img/fn-f3.svg,src/models/pang12/img/fn-f6.svg,src/models/pang12/img/fn-f7.svg,src/models/pang12/img/fn-f9.svg,src/models/pang12/img/led-charging.svg,src/models/pang12/img/led-power.svg,src/models/pang12/img/led-webcam.svg,src/models/pang13/img/fn-f1.svg,src/models/pang13/img/fn-f10.svg,src/models/pang13/img/fn-f11.svg,src/models/pang13/img/fn-f12.svg,src/models/pang13/img/fn-f2.svg,src/models/pang13/img/fn-f3.svg,src/models/pang13/img/fn-f6.svg,src/models/pang13/img/fn-f7.svg,src/models/pang13/img/fn-f9.svg,src/models/pang13/img/led-charging.svg,src/models/pang13/img/led-power.svg,src/models/pang13/img/led-webcam.svg,src/models/pang14/img/fn-f1.svg,src/models/pang14/img/fn-f10.svg,src/models/pang14/img/fn-f11.svg,src/models/pang14/img/fn-f12.svg,src/models/pang14/img/fn-f2.svg,src/models/pang14/img/fn-f3.svg,src/models/pang14/img/fn-f6.svg,src/models/pang14/img/fn-f7.svg,src/models/pang14/img/fn-f9.svg,src/models/pang14/img/led-charging.svg,src/models/pang14/img/led-power.svg,src/models/pang14/img/led-webcam.svg,src/models/pang15/img/fn-f1.svg,src/models/pang15/img/fn-f10.svg,src/models/pang15/img/fn-f11.svg,src/models/pang15/img/fn-f12.svg,src/models/pang15/img/fn-f2.svg,src/models/pang15/img/fn-f3.svg,src/models/pang15/img/fn-f4.svg,src/models/pang15/img/fn-f6.svg,src/models/pang15/img/fn-f7.svg,src/models/pang15/img/fn-f9.svg,src/models/pang15/img/led-charging.svg,src/models/pang15/img/led-power.svg,src/models/pang15/img/led-webcam.svg: convert to Git LFS

@erinxocon erinxocon changed the title src/models/pang12/img/fn-f1.svg,src/models/pang12/img/fn-f10.svg,src/… Migrate svg files to git lfs that were not already there May 27, 2026
@jacobgkau jacobgkau force-pushed the erin/migrate-svg-to-lfs branch from baa18e1 to 5edfc65 Compare May 27, 2026 18:39
@jacobgkau jacobgkau requested a review from a team May 27, 2026 18:42
Copy link
Copy Markdown
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

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

I think I noticed these while preparing #353, but was nervous about rewriting commits going that far back and assumed there'd be no benefit if we weren't rewriting the history (since the full-size files will presumably remain in the history). If migrating them into LFS without rewriting the history would benefit your work, I have no problem with it. (If you think we should rewrite the history, I can also do that; I'd prefer to do that semi-manually to try and keep the original commit timestamps like I did last time.)

The files moved into LFS here are still showing up as expected in the repo.

@erinxocon erinxocon merged commit 431913f into master May 27, 2026
1 check passed
@erinxocon erinxocon deleted the erin/migrate-svg-to-lfs branch May 27, 2026 18:44
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