Monorepo for Tangled โ€” https://tangled.org

appview/repo: split up handlers into separate files #763

merged
opened by oppi.li targeting master from push-qwwtnptuywsl
0
by oppi.li 5 comments
expand 1 commit
appview/repo: split up handlers into separate files

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

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.

sign up or login to add to the discussion
1
by oppi.li 0 comments
expand 1 commit
appview/repo: split up handlers into separate files
2
by oppi.li 0 comments
expand 1 commit
appview/repo: split up handlers into separate files
Labels
refactor
assignee

None yet.

Participants 3
AT URI
at://did:plc:qfpnj4og54vl56wngdriaxug/sh.tangled.repo.pull/3m4xpanyb4r22