-
Notifications
You must be signed in to change notification settings - Fork 24
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
Preview Environment #35
Comments
What about using https://docs.github.com/pages ? |
GitHub Pages only works with Jekyll. We don't use Jekyll |
That's not the information I am having currently:
In the end you just need to serve the static documentation page from the markdown file that has been changed in the PR. |
While it might be possible to use GitHub Pages, I don't see it happening. We wouldn't edit our application code to use GitHub Action aka. a Static Site. The simplest way is most likely to duplicate the deployment and create a second instance on our current server. This wouldn't fulfill the purpose of a temporary review env. So maybe we can set up something with Gitpod or similar. |
If I understand it correctly, the issue is that GitHub Pages still requires a static site, whereas the devdocs are currently a dynamic PHP-based site. |
That's what I'm trying to say, yes |
True, but for the markdown review process you only need the static content of the page that has been modified. Let's say that the markdown for https://devdocs.mage-os.org/docs/main/rest-apis has been changed. |
We have a few custom markdown extensions that'll render special markup. Info boxes, toc, mentions and more are coming. And again, I don't want to add code to the application that solves an infrastructure problem. |
And in the end, it's pure static html, css & maybe js.
Fair enough. Sadly, the community won't be able to contribute because of the lack of access to the infrastructure. Will leave it to it then. Hope to see the feature soon. 😊 |
Reviewer should be able to get a Preview Environment
The text was updated successfully, but these errors were encountered: