Skip to content

[Do not merge] Silly change to demo preview feature #22

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

NicolasGensollen
Copy link
Member

@NicolasGensollen NicolasGensollen commented Feb 17, 2025

This PR is a demo for the HTML preview feature.

It is not meant to be merged!

There are a few points to note:

  • This does NOT work for pull requests opened from forks (so either do not fork the project, or push to the upstream remote instead of the origin remote).
  • There is a first workflow called "Deploy PR previews" that runs on the PR: Once it finishes, it will post a message with a link to the preview. The link will not work immediately. You need to wait for the "deploy" workflow to run in the background before the link actually works (usually takes a couple minutes...).
  • Once this is done, the link points to the preview such that you can inspect how your changes rendered without altering the main website.
  • Once the PR is merged, the preview will be deleted and the modifications will be deployed as the main website.

Copy link

PR Preview Action v1.6.0

🚀 View preview at
https://aramis-lab.github.io/pr-preview/pr-22/

Built to branch gh-pages at 2025-02-17 13:27 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@nburgos
Copy link
Contributor

nburgos commented Feb 26, 2025

This is a super nice tool, thanks!

Do you have an idea why the Job offers page appears all crappy in the preview but not on the published website? I have the same display on the PR #23 for the Job offers and Publications pages and so I'm not sure whether I messed something up or not...

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

Successfully merging this pull request may close these issues.

2 participants