Skip to content

Previews in pull requests #70

Description

@pradyunsg

There's been interest expressed in this. This is me trying to have a dedicated place to track that whole discussion. :)


There are a bunch of options for what to do to get PR previews, to make life easier for reviewers of PRs to this theme.

The ones that I know of are:

  • ReadTheDocs
  • Netlify

Both of these platforms provide free hosting and the ability to preview PRs.

Activity

  1. JulienPalard commented on May 7, 2021

    @JulienPalard
    Member

    I created a netlify account to poke around, but I'm at $DAYJOB, I'll continue poking around later.

  2. pradyunsg commented on May 7, 2021

    @pradyunsg
    MemberAuthor

    Netlify has an open source plan, that this project would qualify for: https://www.netlify.com/legal/open-source-policy

  3. JulienPalard commented on May 8, 2021

    @JulienPalard
    Member

    Netlify asks for the "Act on your behalf" (WTF!?) permission on my github account in order to make it work, I won't give this, so I can't setup netlify (and I don't think anyone else should do, this permission is just plain scary).

    We could create a dedicated low-permission github account to setup netlify, or try something else.

    Would it work with readthedocs? I mean, to test it fully we need to setup language and version switchers, I don't see how it can work.

    About trying something else, it's just a matter of adding an rsync line in the github action, and ... finding a server to host this. @ewdurbin any idea?

    Maybe we could host this on a github page (like python.github.io/python-docs-theme/pr-123/), by pushing to the github page repo from the github action?

    There may be leads around https://github.com/htmlpreview/htmlpreview.github.com too, if it works with js/css (would probably need to massvely rewrite links using sed in the action, leading to other bugs).

    Tried to see if we could build a kind of http proxy around artifacts, like:

    https://example.com/https://github.com/python/python-docs-theme/suites/2680278273/artifacts/59091318/index.html
    

    which would download the zip, cache it, extract it, and provide asked content. But looks like we can't access artifact when not logged in...

  4. Mariatta commented on May 8, 2021

    @Mariatta
    Member

    We discussed adding netlify for the DevGuide here python/devguide#463

    In the end we ended up with using readthedocs preview capabilities.

    I was wondering if we should have an org account on readthedocs. DevGuide's rtd account is currently a personal one.

  5. CAM-Gerlach commented on Dec 16, 2022

    @CAM-Gerlach
    Member

    FYI, the main Python documentation now has Netlify previews, in issue python/cpython#82041 / PR python/cpython#92852 , so there are examples of both platforms in use now for core Python infra.

  6. hugovk commented on Jan 31, 2023

    @hugovk
    Member

    Please see PR #105.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions