-
Notifications
You must be signed in to change notification settings - Fork 61
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
Add Fastly configuration rst #547
Conversation
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 a lot! i have some tweaks
Co-authored-by: David Buchmann <[email protected]>
Co-authored-by: David Buchmann <[email protected]>
Co-authored-by: David Buchmann <[email protected]>
dang, there seems to be quite some code rot in the 2.x branch :-( at leat the spellcheck build is green. i will have a look into the failing tests separately, they have nothing to do with this PR. |
if you could rebase on 2.x branch, the readthedocs build will work again and we can check in the preview if the new documentation page looks right. |
the preview makes me realize: we need to add fastly in the https://foshttpcache--547.org.readthedocs.build/en/547/proxy-configuration.html (the stylesheet seems to be missing in the preview, thats why the page looks a bit strange - it is only applied once the changes are merged) |
Yeah just realized the configuration toctree thing. Already did a push. |
Co-authored-by: David Buchmann <[email protected]>
Co-authored-by: David Buchmann <[email protected]>
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.
awesome, thanks a lot for this contribution!
First version of a fastly configuration example.