Conversation
There was a problem hiding this comment.
Hors MR, ça va "disparaître" une fois que la MR sur la publication d'articles sera publiée.
| token = github.fetch_token( | ||
| TOKEN_URL, | ||
| client_secret=CLIENT_SECRET, | ||
| authorization_response=request.get_full_path(), |
There was a problem hiding this comment.
Je suis étonné du lint pour cette ligne
There was a problem hiding this comment.
J’ai relancé sur l’intégralité du code et ça passe pour moi avec la configuration actuelle.
|
|
||
| ```py | ||
|
|
||
| DECAP_CMS_AUTH = { |
There was a problem hiding this comment.
C'est pas très clair pour moi la manière dont on doit récupérer ce token ? Est-ce une valeur qu'on définit nous-mêmes ? À récupérer quelque part ?
There was a problem hiding this comment.
J’ai ajouté plus de docs.
ludovicdmt
left a comment
ludovicdmt
left a comment
There was a problem hiding this comment.
Bon pour moi, rien de particulier et merci pour la doc.
La question est une question de compréhension.
| - name: Deploy static | ||
| uses: dawidd6/action-ansible-playbook@v2 | ||
| with: | ||
| playbook: showcase.yml | ||
| directory: deploy | ||
| vault_password: ${{secrets.ANSIBLE_VAULT_KEY}} | ||
| options: | | ||
| -l feature -e branch=${{ env.BRANCH_NAME }} | ||
| key: ${{ secrets.SSH_PRIVATE_KEY }} |
There was a problem hiding this comment.
Je ne suis pas sur de comprendre pourquoi on voudrait mettre à jour le site statique quand on merge sur main ?
There was a problem hiding this comment.
Ces lignes spécifiquement permettent d’avoir un site statique à jour quand on est sur une branche. C’est utile parfois, en l'occurrence pour cette MR ça m’était indispensable de pouvoir disposer d’un environnement "feature".
On pourrait éventuellement vouloir désactiver cet environnement (ou le front ou le back) si notre PR n’y touche pas. Mais ça fait un peu de travail et de la complexité pour je pense, pas grand chose.
| - preprod | ||
| - feature |
There was a problem hiding this comment.
C'est un simple site statique donc je ne sais pas si on a besoin d'autant d'env mais bon sait-on jamais et tant que tu en étais là ça semble logique de le faire
There was a problem hiding this comment.
Quand on parle du site statique, je distingue le contenu de la structure (le style, les composants, l'architecture générale). Ici, on parle bien de la structure.
C'est à mon sens beaucoup plus simple pour moi d'avoir un seul workflow par repo. Or actuellement notre workflow c'est du branches −> dev −> main. Ce qui signifie que dès qu'on fait des modifs sur le site statique, ces modifs sont faites sur des branches, puis sur dev puis sur main. Il me semble donc indispensable de pouvoir voir ces mises à jour aux diférentes étapes : étape de la MR (feature), de dev (preprod) et de main (prod).
|
Juste peut être mettre à jour le changelog en prévision de mercredi prochain ? |
|
Mis à jour le changelog. |
3 participants
Specs :
iarbre-showcase-contentsur toutes les branches). En l'état, un nouveau commit suriarbre-showcase-contentne déclenche pas encore d'update du workflow (voir ETQ rédacteur·ice, je veux que mes modifications soient publiées immédiatement #196)