Skip to content

Add docs for AuthenticationClass and TLS#10

Merged
sbernauer merged 8 commits intomainfrom
docs/authN
Apr 11, 2022
Merged

Add docs for AuthenticationClass and TLS#10
sbernauer merged 8 commits intomainfrom
docs/authN

Conversation

@sbernauer
Copy link
Copy Markdown
Member

@sbernauer sbernauer commented Mar 30, 2022

Description

Review Checklist

  • Code contains useful comments
  • (Integration-)Test cases added (or not applicable)
  • Documentation added (or not applicable)
  • Changelog updated (or not applicable)
  • Cargo.toml only contains references to git tags (not specific commits or branches)
  • Helm chart can be installed and deployed operator works (or not applicable)

Once the review is done, comment bors r+ (or bors merge) to merge. Further information

@sbernauer sbernauer requested a review from a team March 30, 2022 08:14
@sbernauer
Copy link
Copy Markdown
Member Author

@soenkeliebau would it be ok for you to ignore the 2 check failures as i don't want to blow up the examples?

Comment thread docs/modules/ROOT/pages/tls.adoc Outdated
Comment thread docs/modules/ROOT/pages/authenticationclass.adoc
Comment thread docs/modules/ROOT/pages/authenticationclass.adoc Outdated
Comment thread docs/modules/ROOT/pages/authenticationclass.adoc Outdated
Comment thread docs/modules/ROOT/pages/authenticationclass.adoc Outdated
Comment thread docs/modules/ROOT/pages/tls.adoc Outdated
Comment thread docs/modules/ROOT/pages/tls.adoc
Comment thread docs/modules/ROOT/pages/usage.adoc
@soenkeliebau
Copy link
Copy Markdown
Member

@soenkeliebau would it be ok for you to ignore the 2 check failures as i don't want to blow up the examples?

I think we should add the missing --- at the beginning of files at least. The other check is useless and we should look into supressing that..

@sbernauer sbernauer merged commit b93ab9e into main Apr 11, 2022
@bors bors Bot deleted the docs/authN branch April 11, 2022 09:02
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.

2 participants