-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
do not automatically download lfs files when cloning/checking out #13449
Conversation
Need to modify travis-scripts to do a git lfs pull before merging this. |
|
.travis.yml
Outdated
@@ -86,6 +86,7 @@ before_install: | |||
|
|||
install: | |||
- git fetch -q | |||
- git lfs pull |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason for adding that here globally? Wouldn't it be enough to add that to tests\travis\install_git_lfs.sh
Guess there is no use in doing that for non UI tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense, will add it there.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
UI tests are still all failing. Seems the images are not downloaded correctly. Haven't tested, but might the order of |
@sgiehl build is working now |
@mattab created simple dev-docs PR here: matomo-org/developer-documentation#265 |
…tomo-org#13449) * do not automatically download lfs files when cloning/checking out * Tweak screenshots readme. * Add lfs pull command. * Update submodule. * Regenerate .travis.yml. * Must use --exclude= in fetch/pull command. * Update submodule to merged master
See git-lfs/git-lfs#2717