search

AcademySoftwareFoundation / rez

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

Address python warnings #1657

Open JeanChristopheMorinPerso opened 9 months ago

JeanChristopheMorinPerso commented 9 months ago

Part of the OpenSSF Best Practices effort.

We have a couple of warnings that we need to fix. We should also make the tests fail if there is any warnings.

BryceGattis commented 7 months ago

@JeanChristopheMorinPerso Can you clarify what needs to be done here? I'm a bit confused.

JeanChristopheMorinPerso commented 7 months ago

@BryceGattis we need to analyze the warnings we see when running the tests. For example, https://github.com/AcademySoftwareFoundation/rez/actions/runs/8483209437/job/23243909858#step:8:354.

parikshittiwari740 commented 7 months ago

Hey @JeanChristopheMorinPerso @BryceGattis, I would like to take this up if nobody is working on it. Seems simple enough for me to start contributing to rez :)

JeanChristopheMorinPerso commented 7 months ago

Sure @parikshittiwari740, feel free to open a PR.

derekhlindell commented 2 months ago

Hello! I'd be interested in picking this one up for Dev days

JeanChristopheMorinPerso commented 2 months ago

Hi @derekhlindell. Feel free to give it a try. But note that rez is not participating to the dez days and as such we might not be able to review your contributions in time. We still appreciate all contributions, so please do contribute if you can. We might be slow to review though.