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

Déploiement automatisé de toutes les branches #54

Closed
wants to merge 8 commits into from
Closed

Conversation

nicolinuxfr
Copy link
Member

J’ouvre la PR, mais ça n’est pas encore au point. J’ai bien un site web pour une branche, exemple avec celle que j’ai créée pour tester : https://dev.vitemado.se/nicolas/

Mais il ne charge que le HTML, ni JS, ni CSS. Les appels aux assets sont en relatif pourtant et j’ai vérifié, tous les fichiers sont bien présents sur le serveur.

Du coup, je sèche un peu. Si quelqu’un veut reprendre, il faudra modifier la condition en tête de l’action (pour le moment j’ai mis uniquement sur ma branche de test, il faudra mettre sur toutes les branches sauf main).

Pour récupérer le nom de la branche, j’ai utilisé cette action clé en main : https://github.com/marketplace/actions/github-slug-action

Le nom de la branche est disponible avec la variable ${{ env.GITHUB_REF_SLUG }}.

@fcamblor
Copy link
Collaborator

Je prends le sujet, je suis trop intéressé par la feature (et j'avais mis les mains dans la config d'urls ;-))

@fcamblor
Copy link
Collaborator

fcamblor commented Apr 12, 2021

Bon, ça fonctionne... le problème vient de ee15cb0 je pense ... la variable BASE_URL transite pas bien jusqu'à npm

Je cherche comment corriger :-)

@fcamblor
Copy link
Collaborator

fcamblor commented Apr 13, 2021

Après pas mal de trial & errors j'ai quelque chose qui fonctionne

Cf commit e9be1a5 :)

Je ferme la PR

@fcamblor fcamblor closed this Apr 13, 2021
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.

None yet

2 participants