Signed-off-by: oppiliappan me@oppi.li
appview/repo: split up handlers into separate files #763
expand 1 commit
hide 1 commit
Signed-off-by: oppiliappan <me@oppi.li>
other than that, I'm 100% supportive to this idea.
I'd prefer we didn't. Go file name convention dictates using an underscore only when needed; I believe the the "handler" bit here is implicit.
perhaps that prefix would make sense if and when we introduce the service layer that we spoke about in earlier conversations.
I don't care much about pascelCase, but I wish we name "handler" explicitly. The repo module itself is already quite ambiguous by mixed with non-handler modules like appview/db.
So we can put the handler prefix to either repo module or each files. And for this case, as the commit message says we are splitting handlers into separate files, so putting prefix for individual files make sense.
I prefer prefix rather than suffix because it makes easier to spot non-prefixed files (like routers or util files) from file explorer.
expand 1 commit
hide 1 commit
Signed-off-by: oppiliappan <me@oppi.li>
expand 1 commit
hide 1 commit
Signed-off-by: oppiliappan <me@oppi.li>
I would like to add
handler_prefix for all of those files. It will make file navigation way easier. Like https://tangled.org/@hailey.at/cocoon/tree/main/server