Monorepo for Tangled tangled.org

orm: extract orm package from appview #875

merged
opened by oppi.li targeting master from op/zxpquyouulmx

includes query and migration helpers

Signed-off-by: oppiliappan me@oppi.li

0
by oppi.li 3 comments
expand 1 commit
orm: extract orm package from appview

This PR is only about code organization right? Seems good to me.

About migration logic, how do you think about using up/down style migrations by maybe using external tools? We can still use our own ORM but migration helpers like goose/v3 or migrate/v4 seems useful.

not entirely convinced yet that something like migrate/v4 adds a lot of value. the onus is still on the developer to write idempotent migrations (i.e., if an up migration adds a column, the down migration should remove it, and the library does not really assist in ensuring this property).

This PR is only about code organization right? Seems good to me.

but yes, this is only for organization, i want to be able to write migrations for knotserver and spindle + be able to use the query builder as well.

pull request successfully merged
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

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