floris-xlx / supabase_rs

Supabase SDK for Rust
MIT License
12 stars 5 forks source link

`lib.rs` should not have all usage examples of this crate #19

Open izyuumi opened 3 months ago

izyuumi commented 3 months ago

I think that the lib.rs module have a lot of documentation at the top, which on docs.rs appears on the main page of this crate. Although it may be useful to have examples to some extent, having most, if not all, examples in the lib.rs creates information overload and is really hard to maintain.

Let me know what you think, I want to hear your opinion before I do anything @floris-xlx :)

floris-xlx commented 3 months ago

nah you're right bro we should probably restructure the entire documentation, the individual examples are good but i agree

floris-xlx commented 3 months ago

since alot of this code was writting in my first week of rust :) i also think alof of the request building code is unnessecarily redunant /boilerplate, we can maybe shave off of a few MS of the response time if we share the client aswell internally in the same authed pool, as supabase just requires the token in the header anyways how it's designed now could be over-engineered