-
Notifications
You must be signed in to change notification settings - Fork 4
Implementar 11ty #13
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
base: master
Are you sure you want to change the base?
Implementar 11ty #13
Conversation
|
Heeej Podem mirar de tancar això? =D |
|
@evilham vas posar que era WIP. Tens interés en continuar amb això per tal de que puguem fer merge? |
Primera fase, el contingut actual està en el layout :-) Closes GeeksCAT#9
Abans d'acceptar hem de canviar "l'output dir" a "_site" Veure: 11ty/eleventy#1455 (comment)
Així també es poden afegir / esborrar fàcilment.
Reimplementa ab7af40
|
Ah, això era al comentari només :-D el WIP li havia tret ja durant aquell dia. Què torneu a fer coses aquest any ;-) Ja està actualitzat amb els últims canvis que vau fer |
|
@evilham moltes gràcies.
Estem movent coses per tornar-hi. |
ifraixedes
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
He deixat uns comentaris d'uns canvis mínims, un cop adreçats farem el merge.
Moltes gràcies per la teva contribució, això ens permetrà ja tenir un sistema que ens permeti fer créixer la web amb més contingut quan en tinguem, mentre que sense això no haguéssim pogut 👏
|
|
||
| on: | ||
| push: | ||
| branches: ["main", "master"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Deixe-m'ho només en master, quan canviem a main, ja canviarem això
| @@ -0,0 +1,16 @@ | |||
| { | |||
| "name": "geekscat-web", | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
afegeix privat: true per evitar que algú pugui publicar-ho accidentalment.
| @@ -0,0 +1,6 @@ | |||
| --- | |||
| layout: layouts/base.njk | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Com que per ara només tenim una pàgina, que és l'índex i el contingut d'aquest fitxer no el llegeix el layout, mou el layouts/base.njk a index.njk i elimina quest fitxer.
Quan tinguem una idea clara de com estructurem el contingut i en comencem a posar-ne, tindrem na visió més clara del layout principal, layouts específics, etc.
|
@evilham Com ho veus per corregir els canvis? Si estas molt liat i necessites uns dies ja en dius, si prefereixes els corregeixo i faig merge, pero preferiria que fossis tu per mantenir el autor del commit inicial per la migració. |
Primera fase, el contingut actual està en el layout :-)
WIP perquè encara falta configurar les GH actions (el dia és llarg)
Closes #9