GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Previews in pull requests · Issue #70 · python/python-docs-theme

Previews in pull requests #70

New issue
Copy link
New issue
Copy link
Closed
#105
Closed
Previews in pull requests#70
#105
Copy link

Description

@pradyunsg
pradyunsg
opened on May 5, 2021
Issue body actions

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
    JulienPalard
    on May 7, 2021
    Member
    More actions

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

    Reacted by Pradyun Gedam
  2. pradyunsg commented on May 7, 2021

    @pradyunsg
    pradyunsg
    on May 7, 2021
    MemberAuthor
    More actions

    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
    JulienPalard
    on May 8, 2021
    Member
    More actions

    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
    Mariatta
    on May 8, 2021
    Member
    More actions

    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
    CAM-Gerlach
    on Dec 16, 2022
    Member
    More actions

    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
    hugovk
    on Jan 31, 2023
    Member
    More actions

    Please see PR #105.

  7. pradyunsg
    closed this as completedin #105on Feb 13, 2023
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

      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.