nordic-rse / nordic-rse.github.io

The community of Research Software Engineers in Nordic countries.
https://nordic-rse.org
MIT License
13 stars 15 forks source link

add preview on pr #445

Closed lucaferranti closed 2 months ago

github-actions[bot] commented 3 months ago

Once the build has completed, you can preview your PR at this URL: https://nordic-rse.github.io/nordic-rse.github.io/previews/PR445/

lucaferranti commented 3 months ago

seems to mainly work (https://github.com/nordic-rse/nordic-rse.github.io/tree/gh-pages/previews/PR445), but does it mess-up the CNAME, since the preview link redirects to nordic-rse.github.io? :thinking:

Maybe the CNAME step should also be run only on main?

cc @rkdarst @bast

bast commented 3 months ago

Maybe the CNAME step should also be run only on main?

Good idea.

lucaferranti commented 3 months ago

closing and reopening to also test the cleanup while updating CNAME

github-actions[bot] commented 3 months ago

Once the build has completed, you can preview your PR at this URL: https://nordic-rse.github.io/nordic-rse.github.io/previews/PR445/

lucaferranti commented 3 months ago

it seems it still redirects to the main website?

bast commented 3 months ago

My feeling is we will need to rename nordic-rse.github.io to something else.

bast commented 3 months ago

To demonstrate my point:

lucaferranti commented 3 months ago

oh I didn't notice the double nordic-rse.github.io in the link!

Wouldn't it be easier to change the comment_pr action to post a correct link

https://nordic-rse.github.io/previews/PR445/

(this works, note the title "Nordic-RSE in a shell" instead of "Nordic-RSE in a nutshell"

lucaferranti commented 3 months ago

closing and reopening for testing

github-actions[bot] commented 3 months ago

Once the build has completed, you can preview your PR at this URL: https://nordic-rse.github.io/previews/PR445/

bast commented 3 months ago

Nice!

lucaferranti commented 3 months ago

@bast @rkdarst this should be working and ready for review

lucaferranti commented 2 months ago

btw @bast @rkdarst this is ready

bast commented 2 months ago

I am sorry I was under the now with notifications and missed this. This is an awesome feature.