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

Typos correction #97

Open
wants to merge 7 commits into
base: dev
Choose a base branch
from
Open

Typos correction #97

wants to merge 7 commits into from

Conversation

sylsta
Copy link

@sylsta sylsta commented Jan 10, 2025

Correction de quelques coquilles. Bravo pour le plugin ! :)

@bchartier
Copy link
Collaborator

Bravo pour le plugin ! :)

Merci beaucoup. Cela fait plaisir.

@bchartier
Copy link
Collaborator

Correction de quelques coquilles.

Merci beaucoup pour cette proposition de contribution au code 👍🏻

Avant de l'analyser plus en détail, nous avons besoin que vous changiez la branche cible de la pullrequest : la préparation des releases se passe dans la branche dev. Nous avons d'ailleurs une autre pullrequest en préparation pour la sortie de v1.0.0 : #95.

Comme il y a des modifications conséquentes entre la branche main et la branche dev, vous risquez de modifier votre code avant de reproposer votre pullrequest.

Votre code propose plusieurs changements :

  • correction de coquilles dans le readme. Ce readme a subi une modification importante. Voir là : Refondre la page readme.md du projet #86. Les modifications sont visibles dans la branche dev et la pullrequest évoquée plus haut. Il est possible que les coquilles en question y soient déjà corrigées. Peut-être pas toutes. Je vous laisse vérifier.
  • ajout de nouvelles sources de données. Idéalement, il faudrait que ce soit discuté avec les autres contributeurs du projet. Je vais voir cela avec eux. Quand vous aurez aligné votre code avec la branche dev, vous verrez que certaines sources de données qui apparaissent comme des modifications de votre PR ne le seront plus (GéoBretagne et GeoPays de Brest).
  • ajout d'une ligne dans le fichier .gitignore. Cette ligne est spécifique à votre configuration. Je ne pense pas que nous l'acceptions. Je vous propose de gérer cela dans votre configuration locale. Regardez cet article : A Personal .gitignore - Even for a Single Repository. J'ai déjà appliqué la solution 3 ; c'est plutôt simple et efficace.
  • changement de version dans le metadata.txt : on va le faire ; ce sera sans doute la 1.0.0.

J'ai bien conscience que l'ensemble de ces remarques peut générer un peu de déception de votre côté. Mais globalement c'est très positif même si je vous demande de retravailler la pullrequest. Merci beaucoup pour votre pullrequest.

Il faudra aussi qu'on vous ajoute aux contributeurs du projet...

@jbdesbas jbdesbas changed the base branch from main to dev January 15, 2025 09:12
@bchartier
Copy link
Collaborator

Bonjour,
Nous n'allons pas intégrer les projets QGIS du SHOM et du BRGM dans un premier temps.
Nous nous rendons compte que nous avons besoin de définir des règles pour ce type de contribution.
Nous allons donc travailler sur la rédaction de ces règles avant toute chose.
J'ai lancé une discussion sur ce sujet dans le canal Matrix du projet (https://matrix.to/#/!DqHgKIoltGIikFRreo:matrix.org). Vous êtes le bienvenu sur ce canal.
J'ai également créé un ticket associé : #99

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants