Conversation
|
I'm certainly in favor of making the standard checks happy :) I did put in most of the legwork, especially when it came to removing vet warnings in generated code. I didn't get it over the finish line as I just lost interest with the pushback: ipld/ipld#92 (comment) I do think it would be unfortunate to have the same standard checks across PL, but not for one of the "flagship core" Go repos, so I think there's value in finishing this. I even helped file a proposal to make the check more lenient, in an attempt to reach some middle ground. |
|
I've done more than my share of style-rage in JavaScript through the past so understand how rusted-on opinions can get when it comes to personal style. There are a few fixes in here that I assume would unambiguously good, such as. the dead code. Having codegen be cleaner for users who are going to It turns out that building a |
FYI Eric and I already agreed on making the generator vet-happy, to unblock the "unified CI" standard checks for other repos. I simply forgot to fix one or two remaining vet errors, because they only triggered in some schema edge cases. |
warpfork
left a comment
There was a problem hiding this comment.
LGTM.
(I still don't care for the struct-initializers-always-explicit-field-names thing, but I've ranted about that before elsewhere, and am giving up on it as not really worth the debate.)
|
I don't reckon this will conflict with anything else open or has been referenced by commit hash anywhere, so I'll just go ahead and rebase+merge this. |
@mvdan @warpfork I know you've had discussions about linting and whatnot and there's strongly held opinions about such things, and I'm wading into this without recollection of your discussions about the path forward (if any) here. It seemed to me that this was low-hanging fruit on the path toward a possible inclusion in the https://github.com/protocol/.github list. There's one minor codegen change in here which is going to add a few more lines to generated code.
Feel free to dismiss this! Not a big deal, just had some muscle memory doing this in a couple of other repos so I thought'd I'd try my hand while I was here.