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

Deployment via github actions #38

Draft
wants to merge 4 commits into
base: source
Choose a base branch
from

Conversation

black-puppydog
Copy link

@black-puppydog black-puppydog commented May 31, 2024

Bonsoir,

J'ai commencé ce travail en regardant le deploy.sh et Makefile just pour faire une toute petite modif (voir le commit doc: ...) mais en le regardant je me suis dit que travis c'est un outil de trop (pour moi en tout cas 😁)

Cela n'est pas complètement fini, mais je voudrais demander si c'est intéressant du tout de faire ce changement. Si oui, je pourrais aussi enlever une parti des choses qui ne servent que pour le workflow travis. :)

Je l'ai déployé aussi ici: https://black-puppydog.github.io/vigilo-website
Évidemment, cela inclut un commit qui modifie l'url de base, et qui n'est pas inclu dans ce PR ici.
Comme le github pages que j'utilise pour tester n'est pas dans la racine de l'url, l'image pour le logo (/images/vigilo.png absolu) ne marche pas. Pour les images dans les pages android j'ai pu changer vers des chemins relatives.

This also brings hugo back up to the latest version, and uses the
upstream of the theme that was being used (the upstream is active,
while the fork no longer works with current hugo)

The `unsafe` option in the renderer is necessary because the shortcodes
we use produce HTML code, and starting with hugo `0.60.0` these would
otherwise just be omitted.z
Since I am deploying to gh-pages, and thus to a non-root path on the domain,
the absolute paths were missing part of the url and were broken.
Luckily, relative paths should be just fine in all our cases. :)
@black-puppydog
Copy link
Author

(déso pour le francais, il est tard et j'ai besoin de sommeil 😆 )

@qhess34
Copy link
Collaborator

qhess34 commented Jun 19, 2024

Très bonne idée, merci pour ta contrib ! J'attends que tu aies finalisé pour valider.

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