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

Update API build and generate 0.19 API docs #3043

Merged
merged 9 commits into from
Nov 18, 2020

Conversation

RichieEscarez
Copy link
Contributor

@RichieEscarez RichieEscarez commented Nov 17, 2020

Related: knative/website#240

Staged: https://5fb326fd6fbbcb00e3428a3a--knative.netlify.app/docs/reference/api/

cc/ @vaikas @lionelvillard

@RichieEscarez RichieEscarez added this to the v0.19.0 milestone Nov 17, 2020
@google-cla google-cla bot added the cla: yes Indicates the PR's author has signed the CLA. label Nov 17, 2020
@knative-prow-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: RichieEscarez

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@knative-prow-robot knative-prow-robot added approved size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. labels Nov 17, 2020
docs/reference/api/README.md Outdated Show resolved Hide resolved

* If you get the
`F0807 13:58:20.621526 168834 main.go:444] type invalid type has kind=Unsupported which is unhandled`
error, the import target might have moved. For more details and an example,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
error, the import target might have moved. For more details and an example,
error, the import target might have moved. For more details, and an example,

I'm bad at punctuation, maybe the above update looks correct ^

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks Mike! Here is the style guide we use for Knative: https://developers.google.com/style/commas

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I've been really busy this week and decided to add the details about the issue i faced this week and quickly dumped details about an issue from the past. I took a second pass at this to better explain. PTAL

@RichieEscarez
Copy link
Contributor Author

RichieEscarez commented Nov 18, 2020

@mpetason if you are okay with the text, can you provide the LTGM so that we can merge this and unblock the 0.19 docs release?

@mpetason
Copy link
Contributor

/lgtm

@knative-prow-robot knative-prow-robot added the lgtm Indicates that a PR is ready to be merged. label Nov 18, 2020
@knative-prow-robot knative-prow-robot merged commit 40776cc into knative:master Nov 18, 2020
RichieEscarez added a commit to RichieEscarez/docs that referenced this pull request Mar 6, 2021
* remove eventing-contrib

* update gen-crd-api-reference-docs version to fix API build errors

* API docs build for 0.19

* revert API version and add troubleshooting docs

* revert file

* remove whitespace

* Update docs/reference/api/README.md

Co-authored-by: Mike Petersen <[email protected]>

* add more detail

* and now fix my typo

Co-authored-by: Mike Petersen <[email protected]>
@RichieEscarez RichieEscarez deleted the 019api branch March 8, 2021 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants