Skip to content

Conversation

@sichanyoo
Copy link
Contributor

Issue #

Description of changes

  • Add links to API reference, dev guide, product page, and examples repo
  • Delete bulk of duplicate content that's already covered by the developer guide

New/existing dependencies impact assessment, if applicable

Conventional Commits

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Copy link
Contributor

@jbelkins jbelkins left a comment

Choose a reason for hiding this comment

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

I wouldn't make reference to the codegen/ content since SDK consumers don't need to interact with it.

Copy link
Contributor

@dayaffe dayaffe left a comment

Choose a reason for hiding this comment

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

Agree with josh, remove ### Codegen Modules from readme and then I am good

@sichanyoo sichanyoo requested a review from jbelkins September 19, 2024 19:16
@sichanyoo sichanyoo merged commit 3069c7a into main Sep 19, 2024
29 checks passed
@sichanyoo sichanyoo deleted the chore/readme-revamp branch September 19, 2024 20: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