supabase / supautils

PostgreSQL extension that secures a cluster on a cloud environment
https://supabase.github.io/supautils
Apache License 2.0
56 stars 12 forks source link

Drop stdlib #41

Closed steve-chavez closed 1 year ago

steve-chavez commented 1 year ago

For discussion, the stdlib is unused and has stagnated.

Not having it makes supautils easier to understand plus makes the docs much shorter, we can just inline the existing functionality on the README. See here for comparison.

I think it's a good idea to keep supautils short/understandable because it's related to security.

Would warrant a v2.0.0 release.

soedirgo commented 1 year ago

👍 agreed - I'd first do a sweep on existing projects on the platform though before rolling this out, just to check if it's used by users somewhere.

steve-chavez commented 1 year ago

@soedirgo Could we pick up this issue again? I'd like to clean up.

Maybe I can just merge it - we don't have to deploy this version yet.

soedirgo commented 1 year ago

Merging it would block future releases until we handle the breaking changes on existing projects. I think we can remove all we can for now, e.g. docs, tests related to stdlib, etc. and then remove the rest on v2.