-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Docs build cleanup #6407
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
Docs build cleanup #6407
Conversation
✅ Deploy Preview for knative ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Confused the build error seems unrelated to your PR /retest |
c461ea2 to
6a69653
Compare
6a69653 to
d1affc2
Compare
This was due to the git clone at Netlify not having fetched any recent branch data; it was all very old ( I do need to figure out why the |
b986ee7 to
fa6dd34
Compare
fa6dd34 to
d83f749
Compare
|
Okay, I think I fixed the worktree stuff. Netlify and "strict" checks should both now pass. |
What was the issue? |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dprotaso, evankanderson 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 |
For some reason, the GitHub action wanted a refspec of |
Fixes lint issues detected in #6405, speeds up docs build
Proposed Changes