Salut, je suis le développeur principal de la version Ruby on Rails du site. Ces dernières 4 ou 5 années, je ne suis plus très actif (mon dernier commit remonte à 6 mois par exemple), donc c'est vraiment bienvenu que d'autres personnes s'intéressent au code et proposent des PR.
Le gros du code a été écrit en 2009 et 2012. Ça remonte pas mal et certaines choses ont mal vieillies (mais moins que ce que l'on pourrait craindre). Dans l'ensemble, Ruby on Rails fait du bon boulot et le code est dans un état correct. Il y a des parties qui ont été écrites avant que Ruby on Rails ne propose une solution à ça, et si on réécrivait ça aujourd'hui, ça serait beaucoup plus simple en utilisant la solution de Ruby on Rails (par exemple, on n'utilise pas ActiveJob pour les jobs de publication des news sur twitter). Mais pour les parties qui ont mal vieillies, je pense plutôt à la doc et aux tests (ou plutôt l'absence de tests).
j'ai bien l'impression qu'il y a des tests automatisés
Non, il n'y en a plus. Il y en a eu par le passé, mais sans intégration continue et avec une couverture loin d'être suffisante. Du coup, ils ne servaient pas à grand chose, et quand ils ont cassé sur des mises à jour de Ruby on Rails ou d'autres gems, la solution la plus simple a été de retirer les tests cassés. À la fin, j'ai fini par tous les retirer.
Au delà du Readme, existe-t-il une aide à la prise en main du code LinuxFr ?
Pas vraiment (ou des trucs trop anciens pour être vraiment utiles), mais j'essaye de répondre aux questions que l'on me pose.
Peut-on proposer une pull request pour une entrée de suivi dont la note est de 0 ?
Oui, c'est possible (surtout s'il n'y a pas de commentaires avec un avis négatif dessus). C'est probablement préférable d'indiquer dans l'entrée de suivi que tu comptes proposer une PR. Globalement, on a plutôt tendance dans l'équipe à accepter facilement les contributions.
Y a-t-il un forum préférable pour poser des questions en lien avec le développement du site LinuxFr ?
Je dirais que le suivi est ce qui s'en rapproche le plus. Il y a eu une liste de diffusion par mails pour ça, mais je crois qu'elle n'existe plus.
Pour proposer une mise à jour du Readme de https://github.com/linuxfrorg/linuxfr.org/, une pull request suffit-elle ou bien est-il préférable d'ouvrir au préalable une entrée de suivi sur LinuxFr ?
Une pull request suffit pour ça.
mysql> CREATE USER 'linuxfr_rails'@'localhost' IDENTIFIED BY 'asecretpassword';
Access denied for user 'linuxfr_rails'@'localhost' (using password: NO)
Dans le fichier config/database.yml, il y a une section development qu'il faut modifier en :
# Les contributions sont les bienvenues
Posté par Bruno Michel (site web personnel) . En réponse au journal Tenter une première modification sur le code de LinuxFr.org. Évalué à 10.
Salut, je suis le développeur principal de la version Ruby on Rails du site. Ces dernières 4 ou 5 années, je ne suis plus très actif (mon dernier commit remonte à 6 mois par exemple), donc c'est vraiment bienvenu que d'autres personnes s'intéressent au code et proposent des PR.
Le gros du code a été écrit en 2009 et 2012. Ça remonte pas mal et certaines choses ont mal vieillies (mais moins que ce que l'on pourrait craindre). Dans l'ensemble, Ruby on Rails fait du bon boulot et le code est dans un état correct. Il y a des parties qui ont été écrites avant que Ruby on Rails ne propose une solution à ça, et si on réécrivait ça aujourd'hui, ça serait beaucoup plus simple en utilisant la solution de Ruby on Rails (par exemple, on n'utilise pas ActiveJob pour les jobs de publication des news sur twitter). Mais pour les parties qui ont mal vieillies, je pense plutôt à la doc et aux tests (ou plutôt l'absence de tests).
Non, il n'y en a plus. Il y en a eu par le passé, mais sans intégration continue et avec une couverture loin d'être suffisante. Du coup, ils ne servaient pas à grand chose, et quand ils ont cassé sur des mises à jour de Ruby on Rails ou d'autres gems, la solution la plus simple a été de retirer les tests cassés. À la fin, j'ai fini par tous les retirer.
Pas vraiment (ou des trucs trop anciens pour être vraiment utiles), mais j'essaye de répondre aux questions que l'on me pose.
Oui, c'est possible (surtout s'il n'y a pas de commentaires avec un avis négatif dessus). C'est probablement préférable d'indiquer dans l'entrée de suivi que tu comptes proposer une PR. Globalement, on a plutôt tendance dans l'équipe à accepter facilement les contributions.
Je dirais que le suivi est ce qui s'en rapproche le plus. Il y a eu une liste de diffusion par mails pour ça, mais je crois qu'elle n'existe plus.
Une pull request suffit pour ça.
Dans le fichier
config/database.yml, il y a une sectiondevelopmentqu'il faut modifier en :