SimVascular / svFSIplus

This repository contains a line-by-line conversion of the Fortran svFSI solver to C++.
Other
6 stars 20 forks source link

Add GitHub Action to check for large files without git lfs #170

Closed mrp089 closed 3 months ago

mrp089 commented 4 months ago

Current situation

Closes #26.

Release Notes

Will fail the pipeline if someone tries to merge a large file (currently set to 1MB) if it's not tracked by git lfs. This hopefully will prevent the chaos behind #149. I will add the action as a branch protection rule for the master.

Documentation

None.

Testing

I'm opening this as a draft to see if the action is executed as intended.

Code of Conduct & Contributing Guidelines

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 57.50%. Comparing base (2fd02ed) to head (4cfdbd6).

:exclamation: Current head 4cfdbd6 differs from pull request most recent head be211a3. Consider uploading reports for the commit be211a3 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #170 +/- ## ========================================== - Coverage 57.70% 57.50% -0.21% ========================================== Files 103 103 Lines 26964 26962 -2 ========================================== - Hits 15560 15504 -56 - Misses 11404 11458 +54 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

mrp089 commented 3 months ago

Giving up on this. I can't get past the error Resource not accessible by integration, which indicates that the Action doesn't have sufficient permissions. I tried: