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

Support for Git-LFS #15

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Support for Git-LFS #15

wants to merge 7 commits into from

Conversation

andreaaletto
Copy link

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

@andreaaletto andreaaletto force-pushed the master branch 2 times, most recently from 13fdf16 to 6132435 Compare June 2, 2023 18:21
@jv-k jv-k reopened this Jun 3, 2023
@jv-k
Copy link
Owner

jv-k commented Jun 5, 2023

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 🍻

@jv-k jv-k added enhancement New feature or request good first issue Good for newcomers labels Jun 5, 2023
@andreaaletto
Copy link
Author

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.
For git lfs feature you can take just the first commit.

@jv-k
Copy link
Owner

jv-k commented Jun 6, 2023

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants