StackStorm / st2-rbac-backend

RBAC backend for StackStorm (previously part of EWC aka StackStorm Enteprise)
https://docs.stackstorm.com/latest/rbac.html
Apache License 2.0
5 stars 12 forks source link

delete pointless requirements parsing code in dist_utils.py #71

Closed cognifloyd closed 1 year ago

cognifloyd commented 1 year ago

All the magic around parsing requirements was entirely unnecessary for this repo as there are NO reqs.

And please! Doing things on import (like raising an error if pip isn't installed) is not ok. Do not have side effects on import, even (and especially) in setup code.

This fix is a follow-up for #70, and is blocking https://github.com/StackStorm/st2/pull/5932 because importing pip breaks generating the lockfile in the latest version of pants (which uses pex, which has a vendored copy of pip).