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

Preview of built docs in the PR #1270

Closed
vfdev-5 opened this issue Sep 8, 2020 · 8 comments
Closed

Preview of built docs in the PR #1270

vfdev-5 opened this issue Sep 8, 2020 · 8 comments

Comments

@vfdev-5
Copy link
Collaborator

vfdev-5 commented Sep 8, 2020

🚀 Feature

Idea is to be able to preview built docs on the CI to simplify PR review.
This issue may be a bit tricky to solve, here some guidelines:

For Hacktoberfest contributors, feel free to ask questions for details if any and say that you would like to tackle the issue.
Please, take a look at CONTRIBUTING guide.

@sroy8091
Copy link
Contributor

sroy8091 commented Oct 3, 2020

@vfdev-5 currently are we using github pages for deploying docs?

@vfdev-5
Copy link
Collaborator Author

vfdev-5 commented Oct 3, 2020

@sroy8091 yes, docs are built and deployed as a Travis CI job on master push: https://github.com/pytorch/ignite/blob/master/.travis.yml#L81

@sroy8091
Copy link
Contributor

sroy8091 commented Oct 3, 2020

We can upload the artifact on build success to s3, including the build number as s3 bucket path. Next step would be to include that link in the pr.

@sroy8091
Copy link
Contributor

sroy8091 commented Oct 3, 2020

Any better approach @vfdev-5 ?

@vfdev-5
Copy link
Collaborator Author

vfdev-5 commented Oct 3, 2020

@sroy8091 I'm eager to do it with netlify like here : beeware/beeware.github.io#426
Let me check it first and conifgure it on netlify side. You can then provide additional info...

@vfdev-5
Copy link
Collaborator Author

vfdev-5 commented Oct 4, 2020

Here is netlify configured:

Currently, it builds the docs as described in CONTRIBUTING and not as in travis (maybe OK).
What remains is to do:

  • add condition to travis yaml to run Docs stage on master only
  • update CONTRIBUTING guide saying that there "netlify/pytorch-ignite-preview/deploy-preview" check available to visualize the docs.

Nice to have:

These scripts can be reused with netlify to build the preview consistent with deployed docs.

@sroy8091
Copy link
Contributor

sroy8091 commented Oct 4, 2020

awesome. will push the travis changes

@vfdev-5
Copy link
Collaborator Author

vfdev-5 commented Feb 6, 2021

Close this as done

@vfdev-5 vfdev-5 closed this as completed Feb 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants