search

AcademySoftwareFoundation / rez

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

Added custom package orderer and fixed package orderer sort logic #1680

Closed isohedronpipeline closed 5 months ago

isohedronpipeline commented 5 months ago

This carries on the work originally started by @pmolodo with the following PRs: https://github.com/AcademySoftwareFoundation/rez/pull/355 https://github.com/AcademySoftwareFoundation/rez/pull/413

This change primarily fixes the issue with package orderers not applying to variants, but also adds a new CustomPackageOrder that can be used to configure a specific ranked list of versions.

codecov[bot] commented 5 months ago

Codecov Report

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

Project coverage is 58.67%. Comparing base (153bd87) to head (bbaf290). Report is 1 commits behind head on main.

:exclamation: Current head bbaf290 differs from pull request most recent head 3a2e3db. Consider uploading reports for the commit 3a2e3db to get more accurate results

Files Patch % Lines
src/rez/package_order.py 91.44% 14 Missing and 9 partials :warning:
src/rez/utils/yaml.py 75.00% 2 Missing :warning:
src/rez/cli/config.py 50.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1680 +/- ## ========================================== + Coverage 58.03% 58.67% +0.63% ========================================== Files 127 127 Lines 17069 17240 +171 Branches 3496 3528 +32 ========================================== + Hits 9906 10115 +209 + Misses 6496 6459 -37 + Partials 667 666 -1 ```

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

isohedronpipeline commented 5 months ago

Closed in lieu of https://github.com/AcademySoftwareFoundation/rez/pull/1684