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

Deploy pull request changes to Netlify so we can preview the devguide #463

Closed
Mariatta opened this issue Feb 12, 2019 · 10 comments
Closed

Comments

@Mariatta
Copy link
Member

Mariatta commented Feb 12, 2019

Original thread in core-workflow mailing list:
https://mail.python.org/archives/list/[email protected]/thread/5FBBRHIWXWRCGHHKCU7WQR3MNFEX4647/

It would be great if we can have a preview of the docs on the internet (without having to manually build it ourselves locally). This is useful for reviewers who want to view the changed documentation.

Service like netlify allows us to deploy the pull request changes for preview.

Before integrating this for docs.python.org, it would be great to try this out in devguide.

cc @matrixise

@Mariatta
Copy link
Member Author

GitMate.io thinks possibly related issues are #133 (Update Devguide about pull request commit documentation.), #104 (Should an issue be created first, before the Pull Request?), #215 (Editing pull request: Can we rebase instead of merge?), #325 (Add tricks for the Pull Requests review?), and python/core-workflow#39 (checking code style for pull request).

@ewdurbin
Copy link
Member

I believe this was successful for pypa/setuptools, as was setup back at PyCon 2018 sprints. Let me know if you need any assistance and I can try to recall what went into that process.

@Mariatta
Copy link
Member Author

I think I got it setup? 😅
test PR #464
https://deploy-preview-464--cpython-devguide.netlify.com/

But I had signed up using a personal account on netlify. Perhaps I should transfer this to you or to the PSF?

@ewdurbin
Copy link
Member

I'm already acting as the Owner for pypa/setuptools with two other administrators.

Happy to accept the transfer and add your user as an Admin. If possible I'd love to recruit one other user for the role.

@Mariatta
Copy link
Member Author

Sounds great! I tried looking into netlify and couldn't really figure out how to transfer or add other admins 😅 Let me know what to do here.

@willingc
Copy link
Collaborator

I'm happy to act as another Admin for the dev guide @Mariatta @ewdurbin unless there is someone else that wishes to do so. I already have the netlify toolchain on my system.

@matrixise
Copy link
Member

@Mariatta I could work on this issue with the doc

@ewdurbin
Copy link
Member

@Mariatta can you share what if any particular settings in netlify were needed to create the initial setup? I can go ahead and create a new project and add the new admins.

@Mariatta
Copy link
Member Author

I went to netlify, and added python/devguide repo.
Then in the build settings:
Build command: sphinx-build -n -W -q -b html -d _build/doctrees . _build/html
Publish directory: _build/html
Branch: master.

I think that's all.

Under Build & deploy > Continuous Deployment > Deploy contexts, just ensure the Deploy previews: "Automatically build deploy previews for all pull requests" is selected (I think it was by default)

Thanks.

@willingc
Copy link
Collaborator

@Mariatta I'm going to close this since it seems to be working well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants