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

Enable git LFS for storing binary files for testing #2159

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

piponazo
Copy link
Collaborator

Currently there are few issues in which we could not progress due to the limitation of adding big files into the reposiroty (see #2042 and #930). I am experimenting with Git LFS which I think could be an interesting solution for being able to upload bigger test data files in our repository (This is useful when we need the original files and not the .exv with the extracted metadata).

I have been following the steps explained here, and it seems to work quite nicely!

At the moment I only tracked the .jpg files in the repository (with the command: git lfs track *.jpg).
One can see the tracked files with: git lfs ls-file
And aparently I could migrate the existing jpg files from normal git objects to LFS ones with git lfs migrate import --include="*.jpg"

It is the first time I work with git LFS so I would appreciate any feedback/suggestions with people with more experience in this topic. So far it looks easy to use.

Note: I had to update all the CI jobs needing to run the tests to indicate that the checkout action should use LFS.

@codecov
Copy link

codecov bot commented Mar 19, 2022

Codecov Report

Merging #2159 (b08aee4) into main (5d5354e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2159   +/-   ##
=======================================
  Coverage   62.65%   62.65%           
=======================================
  Files          97       97           
  Lines       19721    19721           
  Branches     9710     9710           
=======================================
  Hits        12357    12357           
  Misses       5132     5132           
  Partials     2232     2232           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d5354e...b08aee4. Read the comment docs.

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