better-boots-10807
08/31/2022, 7:29 AMbitter-ability-32190
08/31/2022, 10:44 AMflat-zoo-31952
08/31/2022, 12:11 PMbitter-ability-32190
08/31/2022, 1:34 PMbusy-vase-39202
08/31/2022, 5:30 PMbitter-ability-32190
08/31/2022, 5:31 PMJosh and Josh👬 I'd love for us to get more comfy with docstrings in the codebase though. That won't solve all our issues, but does help the newcomer grok certain aspects. I know I could be better about writing docstrings...
busy-vase-39202
08/31/2022, 5:31 PMbusy-vase-39202
08/31/2022, 5:34 PMJosh and JoshFun-ish fact: "Josh C" pronounced aloud sounds like "Joshie". Which is hella confusing in a meeting.
bitter-ability-32190
08/31/2022, 5:34 PMbusy-vase-39202
08/31/2022, 5:34 PMflat-zoo-31952
08/31/2022, 5:35 PMbitter-ability-32190
08/31/2022, 5:35 PMflat-zoo-31952
08/31/2022, 5:37 PMwide-midnight-78598
08/31/2022, 5:38 PMThere are only 2 people in the world I permit to call me "Joshie"@bitter-ability-32190 Is one of them me? That is my last name (without the e)
flat-zoo-31952
08/31/2022, 5:38 PMhelp
) and maybe even just envisioning an ideal developer experience so we know what we're striving forbitter-ability-32190
08/31/2022, 5:39 PMbitter-ability-32190
08/31/2022, 5:39 PMwide-midnight-78598
08/31/2022, 5:42 PMdocumenting its current logic more thoroughlyI think this is big. There are a lot of docstrings and comments in the mainline repo right now, but I dunno - generally just reading it is never enough for me. I end up looking at as many examples in the codebase as possible, to cobble together some understanding of classes I'd never used before. I did mention this before that it was something like 4 hours of reading and reviewing code to implement a 10 minute feature the "Pants way" I think because in our repo, there are multiple ways to achieve the same end goal, and its common in the plugin world to see all of those ways, rather than "the" way... I've mentioned this before, but I wish there was a way to actually strip back a lot of functionality when making a plugin unless we need "advanced" features. Whether that's part of a plugin API, or somehow hidden behind modules/namespaces/types - dunno edit: API "surface area" is basically what I'm describing
flat-zoo-31952
08/31/2022, 5:42 PMbitter-ability-32190
08/31/2022, 5:48 PMI think because in our repo, there are multiple ways to achieve the same end goalThats a problem in itself and it does irk me at night (or in the morning, whenever my son is awake)
better-boots-10807
08/31/2022, 10:40 PMbitter-ability-32190
08/31/2022, 10:52 PMbetter-boots-10807
08/31/2022, 10:52 PMbetter-boots-10807
08/31/2022, 10:52 PMbitter-ability-32190
08/31/2022, 10:53 PMbitter-ability-32190
08/31/2022, 10:59 PMbetter-boots-10807
08/31/2022, 11:00 PMbetter-boots-10807
08/31/2022, 11:00 PM