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

Replace usages of rez.vendor.enum with the built-in enum module #1649

Closed predat closed 6 months ago

predat commented 6 months ago

resolve #1648

JeanChristopheMorinPerso commented 6 months ago

Thanks @predat! Can you sign-off your commit please? See https://github.com/AcademySoftwareFoundation/rez/pull/1649/checks?check_run_id=21673415545 for instructions on how to do it.

Also, am I right to think that enum is used in some of our vendored modules that?

predat commented 6 months ago

Also, am I right to think that enum is used in some of our vendored modules that?

Yes, it’s also used in vendor/pika for exemple

JeanChristopheMorinPerso commented 6 months ago

@predat We can leave rez.vendor.pika untouched. We are planning on reviewing our use of vendored dependencies soon.