Skip to content

Conversation

@timostamm
Copy link
Member

@timostamm timostamm commented Apr 19, 2022

This is the first stab at documentation for the library and code generator. To keep things simple, basically the entire documentation is in the top-level README.md.

The docs are structured a bit similar to the protocol buffer language references (see Java). We go through each protobuf type and show what it's looked like when compiled by us. Then there is a separate section for API docs.

There is also a code example based on the examples in the protobuf repository that can be run without having to install any tooling, so it's easy to get started.

or TypeScript declaration files.

Possible values:
- `target=js` - generates a `_pb.js` file for every `.proto` input file.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should these be separate plugins?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will discuss in #52

- `ts_nocheck=false` - do not print the annotation.


### `eslint_disable`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want to expose this option?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can unexpose, same with the other option. Tracking in #53

`target=ts`.


### `ts_nocheck`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we get away with not exposing this option?

@bufdev bufdev mentioned this pull request Apr 28, 2022
9 tasks
Copy link
Member

@bufdev bufdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we merge this as-is and iterate from there.

@timostamm timostamm merged commit 1b2ae73 into main Apr 28, 2022
@timostamm timostamm deleted the tstamm/add-docs branch April 28, 2022 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants