-
Notifications
You must be signed in to change notification settings - Fork 3
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
Support for Git-LFS #15
base: master
Are you sure you want to change the base?
Conversation
13fdf16
to
6132435
Compare
Update CI jobs
* Remove files not needed * update ci script
Thanks for this, a great idea! I'll isolate the test for it in the CI workflow + minor formatting hygiene and merge it soon. Cheers 🍻 |
Hi, actually I kept on working on my forked master and I have personalized it as per my needs, adding also a new input parameter to enable/disable the creation of a commit. |
Yes, I saw that. The option to disable the commit is a good idea for the scenario where the output file is used for another purpose, eg continuous deployment, and not required to be part of the repo. I'll add it too, thanks. |
In case the .ai files are stored with Git-LFS and the repo is configured with LFS, the CI job throws this error:
\nThis repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting '.git/hooks/post-commit'.\n
meaning that git-lfs bin is not available in the bin PATH of Docker container.
With this PR I'm requesting in the Dockerfile to install also git-lfs
Hoping this could be useful.
Regards,
Andrea