chore: fix spacing of js_library docstrings #961
Conversation
|
it's fixed in our stardoc actually... That's why these things are correct on our docsite. It's stardoc that sucks, our code here was right. |
|
The docsite I keep having to manually fix these strings after slurping & generating 😢 Maybe its not wired up properly in the generate step now? |
|
Ohhh.. I see what the root cause is on the docsite. Some of these docstrings end up with mixed indentations since they are build up from string partials with format 😵💫 In which case, indenting the left on these is the simplest route to avoid the mixed indents. |
| @@ -46,45 +46,45 @@ _ATTRS = { | |||
| "srcs": attr.label_list( | |||
| doc = """Source files that are included in this library. | |||
There was a problem hiding this comment.
nit: when I'm paranoid and de-dent docs to the file level, I start with backslash-newline so the first line is also at column 0
"""\
First line goes here
These docstrings indents are off