-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Improve Serving encryption docs #5955
Conversation
/assign @rhuss |
✅ Deploy Preview for knative ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @ReToCode for adressing. Just a minor nit, but looks good to me and I think it makes it easier for peoples to understand.
You could also use something like callouts, which would render nicely in asciidoc, but could make sense, here, too. Maybe even our HTML rendering framework does support something like this ? 🤔
kind:ClusterIssuer
metadata:
name: cluster-selfsigned-user # <1>
....
....
# <1>: Issuer used by cert-manager to ...
# .... <next line>
# <2>: Knative specific issuer that will uses the CA stored
# in the secret "<name>" which has been created by the
# certificate "<name>"
But that's just some icing on the cake, so I approve it with a /hold
. Feel free to adjust or merge.
/approve
/lgtm
/hold
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ReToCode, rhuss 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 |
@rhuss I don't think it is supported with the config used here: |
/unhold @rhuss PTAL |
@ReToCode even it is not rendered a callouts, the comment syntax in #5955 (review) should stimm work as it is valid YAML commenting. My issue was, that if you add a very long trailing comment at the end of line, it leads to horizontal scrolling in the rendered web pages which is always annoying. So I would at leat put the comment above the line and make it maybe a multi-line comment if it is too long. As this is "just" a usability issue, its fine for me as mentioned (I still add a /hold |
/unhold Let's revisit this as a separate topic. If we do that, we should probably do it as a general pattern :) |
Addressing #5804 (comment)