Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tweak READMEs: #610

Merged
merged 2 commits into from
Apr 20, 2022
Merged

Tweak READMEs: #610

merged 2 commits into from
Apr 20, 2022

Conversation

osa1
Copy link
Member

@osa1 osa1 commented Apr 13, 2022

  • Fix typos

  • Shorten some long lines in markdown files using footnote-style
    markdown links

  • Use # header syntax instead of a mix of # and underscores: it's
    unclear whether --- is smaller than ## or not. Using # for all
    headers is also more consistent and makes it easier to read in
    markdown format (rather than in rendered).

  • Add syntax highlighting to Dart and proto code examples.

  • Split "How to build and use" section into "How to build" and "How to
    use" sections.

  • Move dart pub activate method up and make it the recommended way of
    installing the plugin.

  • Describe how to build standalone native executables for the plugin.

  • Addresses protoc Plugin Build Instructions Clarification #576:

    • For each installation method clarify whether they generate a Dart
      script or standalone native executable.
  • Addresses: Something to suggest about guide #529:

    • Mention that latest version of Dart may be needed. Older versions
      don't have dart pub command and can cause errors when following
      installation instructions.

    • Link to protoc_plugin pub.dev page in protobuf README.

    • Link to Dart installation instructions in protoc_plugin README.

Fixes #576
Fixes #529

- Fix typos

- Shorten some long lines in markdown files using footnote-style
  markdown links

- Use `#` header syntax instead of a mix of `#` and underscores: it's
  unclear whether `---` is smaller than `##` or not. Using `#` for all
  headers is also more consistent and makes it easier to read in
  markdown format (rather than in rendered).

- Add syntax highlighting to Dart and proto code examples.

- Split "How to build and use" section into "How to build" and "How to
  use" sections.

- Move `dart pub activate` method up and make it the recommended way of
  installing the plugin.

- Describe how to build standalone native executables for the plugin.

- Addresses google#576:
  - For each installation method clarify whether they generate a Dart
    script or standalone native executable.

- Addresses: google#529:

  - Mention that latest version of Dart may be needed. Older versions
    don't have `dart pub` command and can cause errors when following
    installation instructions.

  - Link to protoc_plugin pub.dev page in protobuf README.

  - Link to Dart installation instructions in protoc_plugin README.

Fixes google#576
Fixes google#529
@osa1 osa1 requested review from mraleph and sigurdm April 13, 2022 09:27
@osa1
Copy link
Member Author

osa1 commented Apr 13, 2022

@mraleph @sigurdm please feel free to add reviewers.

Copy link
Member

@mraleph mraleph left a comment

Choose a reason for hiding this comment

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

LGTM

protoc_plugin/README.md Outdated Show resolved Hide resolved
protoc_plugin/README.md Outdated Show resolved Hide resolved
protoc_plugin/README.md Outdated Show resolved Hide resolved
@osa1 osa1 requested a review from kevmoo April 19, 2022 18:03
@osa1 osa1 merged commit d232e6e into google:master Apr 20, 2022
@osa1 osa1 deleted the doc_tweaks branch April 20, 2022 07:35
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.

protoc Plugin Build Instructions Clarification Something to suggest about guide
3 participants