WIPACrepo / keycloak-rest-services

Services surrounding KeyCloak, that use the REST API to read/update state
MIT License
1 stars 1 forks source link

Remedy Packaging #18

Closed ric-evans closed 3 years ago

ric-evans commented 3 years ago

So, one thing "special" about keycloak-rest-services is that it isn't really meant to be a python package, but an assemblage of scripts. It happens to have a python library in it (krs) as well.

I'd take suggestions for what to do for this type of thing. We could force it to stick to python package structure, or we could remove the setup.py file. Or something in the middle.

  • @dsschult

I made so could be imported by MOU Dash. This is intended to be a fill-in until the service is up and running. That said, I expect repos with files to be importable.

I vote we remove . The workaround should be in MOU Dash, not the source repo (I can just add a subprocess curl/git). If we want to be importable, then can be re-written to use with as the sole package name.

Edit: MOU Dash only imports

Originally posted by @ric-evans in https://github.com/WIPACrepo/wipac-dev-tools/issues/8#issuecomment-846206326