AcademySoftwareFoundation / rez

An integrated package configuration, build and deployment system for software
https://rez.readthedocs.io
Apache License 2.0
922 stars 329 forks source link

Added configuration option to skip invalid packages #1705

Closed isohedronpipeline closed 3 months ago

isohedronpipeline commented 3 months ago

This PR adds a configuration option to continue solving if an invalid or missing package file is found within a package folder.

Missing package files can happen if the package file itself is in git and the branch is changed. Changing branches does not remove empty directories without a clean operation. This can cause rez to break for any resolve.

Similarly, if any package file contains syntax that is not parseable by your version of python, rez will not resolve any environment at all. This can happen if your package files have f strings and you try to do a resolve in a python 2 environment, for example.

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 42.85714% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 58.23%. Comparing base (3c75a19) to head (6e68a26).

Files Patch % Lines
src/rez/package_resources.py 33.33% 4 Missing :warning:
src/rez/packages.py 57.14% 3 Missing :warning:
src/rez/rezconfig.py 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1705 +/- ## ========================================== - Coverage 58.25% 58.23% -0.02% ========================================== Files 126 126 Lines 17157 17168 +11 Branches 3504 3506 +2 ========================================== + Hits 9995 9998 +3 - Misses 6496 6504 +8 Partials 666 666 ```

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