in general I think we should also value "developer efficiency" (ease of working with, understanding and making changes to the code) for the patterns we use, in addition to "compile time and binary…
LGTM
comments here PeachCloud/peach-workspace#9 (comment)
@glyph cool that you got inspired by this rabbit hole, and thanks for sharing the results in this very scientific way!
these seem like really solid gains, and I just looked through the PR, and…
delta (https://github.com/revoltchat/delta) uses rauth (https://github.com/insertish/rauth) for authentication
however, again this library appears to depend on email.
if we want to ditch the…
This morning I looked into detail in the rocket_auth library https://github.com/tvallotton/rocket_auth
It looks pretty nice, and has an integration with Sqlite, but it expects login to be with an…
with subdirectories, in both templates and in src/routes,
could also consider not having a strict rule that you "need" a subdirectory.
if there is a natural grouping, can put things…
@glyph I will make a short reply, because I agree and don't have much to add. This is also what I was thinking from my browsing around repos.
This is also what they do in the plume repo you…
should this line just be deleted instead of commented out?
I now see the "review" part of gitea, where you can click approve.
@glyph should we continue with the pattern that people merge their own PR after the other approves? I don't see an official "approve code review" in gitea so will just use "lgtm" to say that I've…
@glyph I think this route structure sounds good.
during the summer we were also having a converation about refactoring the peach-web code to follow a pattern more similar to flask blueprints, to…