Faire plus de choses Open Source : rattraper le temps perdu

J'ai été un peu occupé ces derniers temps et, malheureusement, je n'ai pas eu beaucoup de temps pour contribuer à l'open source. J'avais hâte de revenir sur la bonne voie, mais j'ai rencontré quelques problèmes. Dans mon dernier article, j'ai expliqué comment j'étais prêt à plonger dans Mattermost Mobile et à implémenter la même fonctionnalité sur laquelle j'avais travaillé précédemment pour l'application Web - en ajoutant un paramètre pour basculer le rendu des émoticônes (:D) sous forme d'émojis (?) - mais j'ai dû abandonne très vite cette idée. Il s'avère que WSL (Windows Subsystem for Linux) n'est pas si génial pour le développement mobile.

"Ça ne pourrait pas être si difficile..."

Le développement pour Mattermost n'est officiellement pris en charge que pour MacOS et Linux. Ce n'était pas un problème lorsque l'on travaillait sur la webapp. Pendant que je travaillais sur l'application Web, j'ai pu utiliser WSL et tout a parfaitement fonctionné. Mais lorsque j'ai essayé de commencer à travailler sur l'application mobile, j'ai rencontré problème après problème.

En suivant les instructions de configuration de l'environnement de développement pour Linux, j'avais besoin de configurer un émulateur Android, mais essayer d'en exécuter un dans WSL n'a pas fonctionné - il ne détecterait pas l'émulateur. Je ne pouvais pas non plus développer sous Windows car le script de pré-installation exécutait un script .sh. J'ai donc passé quelques jours à essayer de me connecter depuis WSL à l'émulateur Android fonctionnant sous Windows, en vain - il n'apparaissait tout simplement pas dans la liste des émulateurs dans WSL. J'ai passé des heures à parcourir les messages du forum et l'essentiel de GitHub, mais il semblait que cela n'allait tout simplement pas fonctionner.

J'ai également essayé de transférer la connexion USB de mon propre appareil vers WSL pour essayer de développer sur mon appareil. Apparemment, usbipd-win devrait faire fonctionner cela, comme mentionné dans cet article sur Microsoft Learn, et pourtant, chaque fois que je réessayais de transférer mon appareil vers WSL, la liste des appareils connectés devenait vide.

J'ai cherché dans l'historique des messages dans le chat officiel des contributeurs de Mattermost, mais tout ce que j'ai trouvé, ce sont des messages d'autres personnes aux prises avec la même chose. J'ai donc pensé que la dernière chose que je pouvais essayer était de créer mon propre message et de demander de l'aide, ce à quoi, heureusement, quelqu'un a répondu pour essayer d'aider, mais malheureusement, il ne faisait pas partie de l'équipe mobile et m'a demandé d'essayer de demander dans le mobile. le chat de l'équipe, où je n'ai pas encore reçu de réponse.

Malheureusement, dans l'immédiat, il semble que je ne pourrai pas travailler sur le sujet.

Essayer autre chose

En attendant, j'ai cherché à aborder d'autres problèmes. En recherchant les problèmes intitulés « aide recherchée », j'ai trouvé ce problème pour la CLI GitHub :

`gh run view` devrait lister les branches entre crochets #10038

SacCrapaud publié le 07 décembre 2024

Décrivez le bug

gh run view répertorie les branches entre parenthèses, mais je pense qu'il devrait les lister entre crochets pour s'aligner sur les directives de Primer :

Afficher les noms des branches entre parenthèses et/ou cyan

Étapes pour reproduire le comportement

gh vue d'exécution

Comportement attendu ou réel

Les invites d'affichage d'exécution gh doivent afficher les branches entre crochets.

Journaux

❯ gh run view
? Select a workflow run  [Use arrows to move, type to filter]
> - Verifying attestations offline fails, Discussion Triage (trunk) 4h55m1s ago
  - Decoding, Discussion Triage (patch-1) 4h59m32s ago
  ✓ Decoding, PR Automation (patch-1) 4h59m43s ago
  ✓ Issue Automation, Issue Automation (trunk) 5h20m31s ago
  - `gh repo rename myorg/newname` results in `myorg/myorg-newname`, Discussion Triage (trunk) 10h13m50s ago
  - 401 Error at every turn, Discussion Triage (trunk) 10h15m20s ago
  - 401 Error at every turn, Discussion Triage (trunk) 10h15m20s ago

Voir sur GitHub

Cela semblait être un changement assez simple et je pensais que ce serait une belle pause par rapport aux problèmes colossaux que j'avais entrepris - quelque chose pour me remettre sur un rythme décent.

J'ai été satisfait de la simplicité de la configuration du développement. J'avais déjà installé Go, donc tout ce que j'avais à faire était de compiler le binaire.

J'ai lu les documents de contribution qui ont également été très utiles et m'ont aidé à commencer à déterminer où trouver le code pertinent. J'ai passé un peu de temps à le parcourir et j'ai trouvé la fonction chargée d'afficher les noms de branches dans la commande d'exécution. Je n'avais jamais vraiment écrit Go auparavant, mais cela rappelait beaucoup printf() de C. J'ai effectué le changement et ouvert un PR, mais dans l'excitation, j'ai oublié de mettre à jour les tests, ce que m'a rappelé un critique.

correctif : répertorier les branches entre crochets dans `gh run view` #10043

ouday-rana publié le 08 décembre 2024

Corrections #10038

Modifications

pkg/cmd/run/shared : remplacement des parenthèses dans selectRun() par des crochets.

Avant

func selectRun() {
  ...
  fmt.Sprintf("%s %s, %s (%s) %s", ...)
  ...
}

Après

func selectRun() {
  ...
  fmt.Sprintf("%s %s, %s [%s] %s", ...)
  ...
}

Voir sur GitHub

J'ai fouillé les tests, identifié ceux qui échouaient et les ai mis à jour. Malheureusement, un responsable m'a ensuite informé qu'il était en fait toujours en train de trier ce problème et qu'il avait appliqué de manière préventive l'étiquette « aide recherchée », et que mon PR ne pouvait pas être fusionné. J'étais un peu déçu, mais je pensais que j'avais fait ma part donc ce n'était pas grave.

Commentaire pour #10038

SacCrapaud a commenté le 09 décembre 2024

? Hé @uday-rana, merci de l'intérêt que vous portez à cela !

J'aimerais attendre que ce problème soit réglé par notre premier intervenant en rotation cette semaine.

J'étais probablement trop impatient en ajoutant le label Help Wanted - Je veux toujours que notre premier intervenant fasse le tri pour s'assurer que nous sommes d'accord sur le travail avant de le mettre en œuvre ❤ ?

Je ne prévois aucun problème, mais je vous le fais quand même savoir pour que vous sachiez ce que nous attendons avant que l'un de nous examine le PR. Si notre premier intervenant est d'accord, alors nous accepterons volontiers un PR ?

Désolé pour la confusion ! À l'avenir, je n'ajouterai pas ces étiquettes plus tôt - cela a involontairement ignoré notre processus de tri ?

Voir sur GitHub

Deuxième tour

En parcourant GitHub pour trouver d'autres problèmes sur lesquels travailler avec l'étiquette « aide recherchée », j'ai trouvé encore un autre problème Mattermost, cette fois pour l'application Web. Comme j'avais déjà configuré l'environnement de développement pour la webapp, j'ai pensé que je pourrais rapidement passer à l'action.

Ajouter un horodatage aux noms de fichiers d'images collés pour les différencier #29524

matièremod publié le 09 décembre 2024

Le problème

Lorsque vous collez une image du presse-papiers dans la saisie du message, elle obtient le nom de fichier image.png. Si plusieurs images sont collées, elles obtiennent toutes le même nom de fichier image.png. Cela peut poser un problème lors du téléchargement de chaque fichier portant le même nom avec risque d'écrasement.

Solution proposée

Générez automatiquement un nom de fichier incluant l'horodatage avec ce format :

image-2024-11-18-6-29-57-PM

Thème le plus important : https://hub.mattermost.com/private-core/pl/xryg3tedg3bbxq3xuwnrj9ymyc

---

Si vous êtes intéressé, veuillez commenter ici et venir rejoindre notre chaîne communautaire « Contributeurs » sur notre serveur de build quotidien, où vous pourrez discuter de questions avec les membres de la communauté et l'équipe principale de Mattermost. Pour des conseils techniques ou des questions, veuillez rejoindre notre chaîne communautaire "Développeurs".

Nouveaux contributeurs, veuillez consulter notre Guide du développeur.

JIRA : https://mattermost.atlassian.net/browse/MM-62003

Voir sur GitHub

J'étudie ce problème au moment de la rédaction de cet article de blog et j'ai identifié la source du problème. Lorsque vous publiez une image à partir du presse-papiers dans un navigateur ou un environnement de type navigateur (à la manière de l'application Electron), si l'image n'a pas de nom, le navigateur lui donnera automatiquement le nom "image.png". J'ai testé cela avec Firefox et Edge et cela est vrai pour les deux. Pourquoi est-ce un problème ? L'application dispose déjà d'une logique qui ajoute des horodatages aux fichiers sans nom. Mais comme ces images reçoivent automatiquement des noms lors du collage à partir du presse-papiers, les horodatages ne sont pas définis.

La solution la plus simple serait simplement de coder en dur un chèque pour le nom « image.png ». Mais je me demande que se passe-t-il si l'image n'est pas au format PNG ? Est-ce même possible dans le presse-papiers ? Et que se passe-t-il si l'utilisateur télécharge un fichier réellement nommé « image.png » ? C'est quelque chose dont je devrai discuter avec les développeurs.

Quoi qu’il en soit, je suis déterminé à résoudre ce problème. J'espère que tout se passera bien.

Ce qui précède est le contenu détaillé de. pour plus d'informations, suivez d'autres articles connexes sur le site Web de PHP en chinois!