-
Notifications
You must be signed in to change notification settings - Fork 117
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
Add validate-lfs.sh check from Android. #203
Conversation
Generated by 🚫 Danger Swift against f00bcb7 |
115e427
to
f00bcb7
Compare
Kudos, SonarCloud Quality Gate passed! |
📱 Scan the QR code below to install the build for this PR. If you can't scan the QR code you can install the build via this link: https://i.diawi.com/Aq6Hdn |
Codecov ReportBase: 11.97% // Head: 11.97% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## develop #203 +/- ##
========================================
Coverage 11.97% 11.97%
========================================
Files 208 208
Lines 14146 14146
Branches 6844 6844
========================================
Hits 1694 1694
Misses 12412 12412
Partials 40 40 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
Good stuff although I was hoping we could have it as a git pre-commit hook as well. Probably not that important, this is not something we'll encounter very often
Sadly this would work the same way as LFS and only be available for whoever added the hook (at which point they should just |
Copied across from the EA workflow and script.
Will test separately once merged with a fresh clone that didn't
git lfs install