build(bindgen): check for corresponding .zig file #15896
Conversation
| @@ -1,5 +1,9 @@ | |||
| // This is the public API for `bind.ts` files | |||
| // It is aliased as `import {} from 'bindgen'` | |||
| /** | |||
There was a problem hiding this comment.
does jsdoc make this a comment for the file or the next declared symbol (Type)? i could be totally wrong and this form is correct.
There was a problem hiding this comment.
for //? no. It should make one for /** */ but I'm not seeing it. Maybe there's a problem with the tsconfig?
src/codegen/bindgen-lib.ts
Outdated
| @@ -196,6 +200,23 @@ export namespace t { | |||
| export const ByteString = builtinType<string>()("ByteString"); | |||
| /** | |||
| * DOMString but encoded as `[]const u8` | |||
| * | |||
| * @example | |||
There was a problem hiding this comment.
i think simple examples like these aren't worth their line count, especially when my laptop screen can only fit 70 lines of code at once. this is why i try to write extremely concise comments.
There was a problem hiding this comment.
It makes it appear nicely when hovering over the symbol.
If this was ts-only, VSCode would still perform syntax highlighting.
There was a problem hiding this comment.
Ah the hover doesn't appear in screenshots :(
When generating bindings for
foo.bind.ts, ensure afoo.zigfile exists next to it. Builds will fail downstream if this is missing, this error just makes it surface sooner and in a more clear manner.