-
Notifications
You must be signed in to change notification settings - Fork 690
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
site: point to latest github release tag where possible #1757
site: point to latest github release tag where possible #1757
Conversation
7d262e8
to
36e29c8
Compare
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.
I marked a few links that 404. If those will be fixed by a later PR, feel free to dismiss the comments and land this :)
@@ -13,7 +13,7 @@ If you don't have a cluster with that capability see the [Running without a Kube | |||
### Recommended installation details | |||
|
|||
The recommended installation of Contour is Contour running in a Deployment and Envoy in a Daemonset with TLS securing the gRPC communication between them. | |||
The [`contour` example]({{site.github.repository_url}}/tree/master/examples/contour/README.md) will install this for you. | |||
The [`contour` example]({{site.github.repository_url}}/blob/{{ site.github.latest_release.tag_name }}/examples/contour/README.md) will install this for you. |
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.
This resolves to https://github.com/projectcontour/contour/blob/v0.15.3/examples/contour/README.md
which is 404.
@jpeach yup, all those links won't be valid today. This PR cannot land til 1.0 is cut. |
2767bce
to
b83c2ff
Compare
Fixes projectcontour#1619 For 1.0 the plan of record is to keep the examples on GitHub. This is due to the difficulty in doing directory index style static resources from projectcontour.io. To support this weave the name of the current github release through the website. Note that at the present, in the rc.2 timeframe, this will break a lot of links so this PR is targeted for 1.0 (or shortly after) which will bring the guides, resources, and release documentation in line with the latest released tag on GitHub. Signed-off-by: Dave Cheney <[email protected]>
b83c2ff
to
5e430d4
Compare
Fixes #1619
For 1.0 the plan of record is to keep the examples on GitHub. This is
due to the difficulty in doing directory index style static resources
from projectcontour.io.
To support this weave the name of the current github release through the
website.
Note that at the present, in the rc.2 timeframe, this will break a lot
of links so this PR is targeted for 1.0 (or shortly after) which will
bring the guides, resources, and release documentation in line with the
latest released tag on GitHub.
Signed-off-by: Dave Cheney [email protected]