TSM-06: convert Group A services to TypeScript - #563
TSM-06: convert Group A services to TypeScript#563Justin Hammond (Justintime50) wants to merge 15 commits into
Conversation
ea893f6 to
66869ff
Compare
66869ff to
8a875f7
Compare
8a875f7 to
bf2cfcf
Compare
bf2cfcf to
7bab25e
Compare
7bab25e to
ca4ccd5
Compare
ca4ccd5 to
92acfde
Compare
44b5d7b to
f1023c7
Compare
Alex Matthews (ralexmatthews)
left a comment
There was a problem hiding this comment.
Hmmm... this looks like its more of a "just adding types for the sake of adding types" and less like implementing typescript checking. Lots of unknowns, anys, and general type omissions. I see lots of places where we introduce more any's, and while we technically have "explicit parameter and return types", I don't think most of them are actually helpful. So many are just
(params: Record<string, unknown>): Promise<unknown>
I might not be getting the point of this, and they are typed in the ./types/ directory. I don't know if we want to implement those in the source directly instead, or treat those as the source of truth or what, but it feels divergent.
59c44ae to
370832b
Compare
370832b to
79775e7
Compare
Alex Matthews (ralexmatthews)
left a comment
There was a problem hiding this comment.
Niiice. Still a handful of anys thrown around, but hoping those get fixed later
Summary
Converts Group A service modules to TypeScript and applies minimal typed compatibility fixes needed for the current base-service conversion, with runtime behavior preserved.
Changes in this PR
Testing
Notes