rez icon indicating copy to clipboard operation
rez copied to clipboard
verified publisher icon AcademySoftwareFoundation

Address python warnings

Open JeanChristopheMorinPerso opened this issue 1 year ago • 8 comments

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.

  • https://www.bestpractices.dev/en/projects/8389#warnings
  • https://www.bestpractices.dev/en/projects/8389#warnings_fixed
  • https://www.bestpractices.dev/en/projects/8389#warnings_strict
  • https://www.bestpractices.dev/en/projects/8389?criteria_level=1#warnings_strict

JeanChristopheMorinPerso avatar Feb 17 '24 19:02 JeanChristopheMorinPerso

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

BryceGattis avatar Apr 04 '24 01:04 BryceGattis

@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.

JeanChristopheMorinPerso avatar Apr 06 '24 14:04 JeanChristopheMorinPerso

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 :)

parikshittiwari740 avatar Apr 11 '24 17:04 parikshittiwari740

Sure @parikshittiwari740, feel free to open a PR.

JeanChristopheMorinPerso avatar Apr 13 '24 18:04 JeanChristopheMorinPerso