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

Ensure https is shown in code block in tutorial #661

Merged
merged 1 commit into from
May 17, 2022

Conversation

chloesoe
Copy link
Contributor

@chloesoe chloesoe commented May 13, 2022

Summary

Having an URL in a code block without \https will be rendered as a link in a browser and not showing https://; thias leads to errors when copy pasting to the CLI while running the tutorial.

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking,
    as they show up in the changelog
  • Update the documentation.

@chloesoe chloesoe requested a review from a team as a code owner May 13, 2022 13:11
@chloesoe chloesoe requested review from simu and tobru and removed request for a team May 13, 2022 13:11
@chloesoe
Copy link
Contributor Author

looks like I don't have enough rights to set a label. The label should be documentation, thanks for adding

@ccremer ccremer added the documentation This issue or pull request affects documentation label May 17, 2022
@ccremer
Copy link
Contributor

ccremer commented May 17, 2022

@chloesoe Can you amend your commit so that DCO passes?

@ccremer ccremer changed the title Ensure https is shown in code block in documentation Ensure https is shown in code block in tutorial May 17, 2022
@chloesoe chloesoe force-pushed the fix-url-to-apply-crd branch from 68be311 to 2f2e7a9 Compare May 17, 2022 07:25
Having it without `\https` it will be rendered as a link and not showing
`https://` which leads to errors when copy pasting.

Signed-off-by: Daniel Hauswirth <info@chloesoe.ch>
Signed-off-by: chloesoe <info@chloesoe.ch>
@chloesoe chloesoe force-pushed the fix-url-to-apply-crd branch from 2f2e7a9 to a215609 Compare May 17, 2022 07:31
@ccremer ccremer enabled auto-merge May 17, 2022 07:33
@ccremer ccremer merged commit 6f7e0f4 into k8up-io:master May 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation This issue or pull request affects documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants