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

fixed document links #1037

Merged
merged 3 commits into from Apr 20, 2019

Conversation

Projects
None yet
3 participants
@akosmaroy
Copy link

commented Apr 6, 2019

fixes some broken document links noted in #1036

Akos Maroy

@dasrecht dasrecht self-assigned this Apr 10, 2019

@dasrecht
Copy link
Member

left a comment

Hi @akosmaroy - Thanks for your contribution!

I shortly tested it and the links seem to have been more broken than anticipated. You can adapt the links to link directly to the markdownfile.md#anchor

E.g. [`.lagoon.yml`](lagoon_yml.md#environmentsnamerollouts)

Feel free to update the PR and I'll merge it in after testing.

Akos Maroy
@akosmaroy

This comment has been minimized.

Copy link
Author

commented Apr 12, 2019

updated: removed './' prefix for local directory references, added '.md' postfix where it was missing

@dasrecht
Copy link
Member

left a comment

Looks good! Thanks a lot @akosmaroy

@dasrecht dasrecht merged commit af0d0d7 into amazeeio:master Apr 20, 2019

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details

@Schnitzel Schnitzel added this to the v0.23.0 milestone May 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.