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

ci: vérification entêtes markdown #1143

Merged
merged 12 commits into from
Jul 11, 2024
Merged

Conversation

gounux
Copy link
Contributor

@gounux gounux commented Jun 22, 2024

Ajout de la vérification de l'entête markdown

Utilisable en utilisant le geotribu_cli v >= 0.34.3

@github-actions github-actions bot added outillage Linter, git hooks, CI, CD, build, compilation, devops, intégration et déploiement continus, etc. dependencies Pull requests that update a dependency file articles Tutoriels, actualité, billet... labels Jun 22, 2024
@gounux gounux marked this pull request as draft June 22, 2024 08:10

This comment has been minimized.

1 similar comment

This comment has been minimized.

@Guts
Copy link
Member

Guts commented Jun 22, 2024

@gounux tu as repoussé le tag sur https://github.com/geotribu/cli/releases/tag/0.34.0 non ?

@Guts
Copy link
Member

Guts commented Jun 22, 2024

@gounux tu as repoussé le tag sur https://github.com/geotribu/cli/releases/tag/0.34.0 non ?

Je me suis permis de faire le ménage en supprimant les 2 drafts de relases créés, le tag qui avait été repush et j'ai remis un tag 0.34.0 tout neuf avec les dernières corrections, notamment celle où tu rétablis la CI/CD. D'ailleurs bravo, c'est bien mon honteux copié/collé qui avait foutu le dawa !

Fais un bon coup de git pull --tags --prune de ton côté je pense.

@gounux
Copy link
Contributor Author

gounux commented Jun 22, 2024

@Guts merci !

Oui je vois que le job vers PyPi est passé (https://pypi.org/project/geotribu/ v0.34.0 🚀 )

This comment has been minimized.

1 similar comment

This comment has been minimized.

This comment has been minimized.

@Guts Guts force-pushed the ci/check-article-yaml-headers branch from b922d8e to a3ea4ef Compare July 3, 2024 10:01

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@gounux gounux marked this pull request as ready for review July 3, 2024 12:33
@gounux
Copy link
Contributor Author

gounux commented Jul 3, 2024

@Guts le premier job est passé ! 🚀

Comment tu vois le truc ? On le configure sur tous les articles passés avec un wildcard genre content/articles/**/*.md ?

@Guts
Copy link
Member

Guts commented Jul 3, 2024

@Guts le premier job est passé ! 🚀

Il n'y a aucune erreur ou warning ? Top, bravo !

Comment tu vois le truc ? On le configure sur tous les articles passés ? Ou seulement depuis la ressurection de 2020 ?

Clairement, GO sur les 202*.

Prochaine évolution, ça pisse un rapport Markdown qu'on envoie en commentaire de la PR si code de sortie > 0

@gounux
Copy link
Contributor Author

gounux commented Jul 3, 2024

Prochaine évolution, ça pisse un rapport Markdown qu'on envoie en commentaire de la PR si code de sortie > 0

super idée !

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

@gounux
Copy link
Contributor Author

gounux commented Jul 9, 2024

cc @Guts il y a l'air d'y avoir pas mal d'articles dont le ratio de l'image n'est pas autour de 1.5

À mes yeux ce sont des modifs structurantes qui touchent au découpage de l'image et donc à la nature du contenu. On augmente l'intervalle de ratio pour le mettre entre disons 1 et 2 ? Le souci serait que du coup cette vérif ne servirait plus à grand chose

@Guts
Copy link
Member

Guts commented Jul 9, 2024

cc @Guts il y a l'air d'y avoir pas mal d'articles dont le ratio de l'image n'est pas autour de 1.5

Ah mince, tant que cela ?

À mes yeux ce sont des modifs structurantes qui touchent au découpage de l'image et donc à la nature du contenu.

On parle de l'image d'en-tête qui n'est même pas affichée sur le contenu même donc bon c'est pas non plus lié au fond, ça reste de la forme.

Sortons la liste et regardons. Ajuster le ratio me paraît en effet être la bonne solution.

Copy link
Contributor

👏 Le markdown ajouté ou modifié est impeccable ! Merci 🥇

Copy link
Contributor

🎉 Le site de prévisualisation pour cette PR a été généré et est disponible sous ce lien.
Commit de référence : d773f12.

@gounux gounux merged commit da12626 into master Jul 11, 2024
7 checks passed
@gounux gounux deleted the ci/check-article-yaml-headers branch July 11, 2024 21:20
@github-actions github-actions bot temporarily deployed to pr-previews July 11, 2024 21:20 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
articles Tutoriels, actualité, billet... dependencies Pull requests that update a dependency file outillage Linter, git hooks, CI, CD, build, compilation, devops, intégration et déploiement continus, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants