tangled.org / core
Monorepo for Tangled — https://tangled.org
fork atom

appview: remove ResolvedRepo for eternity #821

merged
opened by boltless.me targeting master from sl/yurolxtlpsmz
STACK 11
#821 appview: remove ResolvedRepo for eternity
1
#2
#820 appview: remove use of all fields under ResolvedRepo
0
#2
#812 appview: prefer models.Repo over ResolvedRepo internally
0
#3
#811 appview/pages: fix pages.RepoComparedDiffFragment()
0
#4
#810 appview: use less RepoInfo for ResolvedRepo
0
#4
#809 appview: remove ResolvedRepo.RolesInRepo()
0
#4
#808 appview: diet the reporesolver
0
#4
#807 appview: remove unused fields on RepoInfo
0
#4
#802 appview: use less OwnerSlashRepo() in handlers
0
#5
#801 appview/pages: resolve did on render
0
#5
#806 appview/db: remove unused struct
0
#4
0
by boltless.me 1 comment
expand 1 commit
e5fcb607
appview: remove ResolvedRepo for eternity
boltless.me

@oppi.li (testing the at-mention) this stack is ready for review. basically all I did here are:

  • diet the repoResolver so we can switch to repo service later (see #800)
  • remove ResolvedRepo

Because the dependencies are quite tangled and ResolvedRepo was used all over the place, I had to divide the change in multiple steps and migrated gradually.

sign up or login to add to the discussion
1
by boltless.me 1 comment
expand 1 commit
c28eb3c3
appview: remove ResolvedRepo for eternity
oppi.li

entire stack lgtm, thanks for the hard work on this!

sign up or login to add to the discussion
2
by boltless.me 1 comment
expand 1 commit
cdc4ebf7
appview: remove ResolvedRepo for eternity
oppi.li

entire stack lgtm, thanks for the hard work on this!

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:xasnlahkri4ewmbuzly2rlc5/sh.tangled.repo.pull/3m5t2gtxrpi22