Alex Eagle
Alex Eagle
Hey, sorry I disappeared from the repo for a bit - what's next for this one?
To be precise I think devx teams will commonly install under another name, so we should use $0 everywhere and/or a product_name config setting. Some might actually install as 'bazel'...
We think this already works. @f0rmiga will take one more stab at validating that it already works.
Maybe just add fix-visibility plugin to something that's already reporting to buildbuddy.
FYI @c-parsons could use someone's help to review/merge :)
Yeah I considered changing the existing DocstringParser class, but of course it has a bunch of fields which are specific to docs with structured sections like `Args:` and therefore it's...
My question for @tetromino is whether you have time to review a big refactoring here and if that's what you want, before I spend time digging in to how is...
@tetromino I think the ball is still in your court, I don't want to invest in a refactoring here that doesn't get merged
Thanks for the review! I'm not sure when I'll get time to return to this, but I'll try to get someone to carry it to the finish.