Skip to content

Contributing.rst#1848

Merged
jeanas merged 22 commits into
3.10from
unknown repository
Apr 28, 2022
Merged

Contributing.rst#1848
jeanas merged 22 commits into
3.10from
unknown repository

Conversation

@ghost

@ghost ghost commented Apr 13, 2022

Copy link
Copy Markdown

Relisant en profondeur le fichier de contribution, je me suis rendu compte que la structure de celui-ci était curieuse. Je me permet donc de proposer une petite modification, qui reste à fignoler.

@christopheNan christopheNan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci pour ce nouveau regard qui nous fait progresser.

Comment thread CONTRIBUTING.rst Outdated
Comment thread CONTRIBUTING.rst Outdated
Comment thread CONTRIBUTING.rst Outdated
Comment thread CONTRIBUTING.rst
Comment thread CONTRIBUTING.rst Outdated
Comment thread CONTRIBUTING.rst Outdated
Comment thread CONTRIBUTING.rst Outdated
th71 and others added 7 commits April 14, 2022 16:15
Intégration de suggestion.
Reste quelques points que je dois reprendre avant de valider.

Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

@ghost ghost left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remarques prises en comptes.
La structure globale me semble encore un peu bancale.

Comment thread CONTRIBUTING.rst Outdated

@christopheNan christopheNan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cela me va très bien. juste un petit pinaillage.

Comment thread CONTRIBUTING.rst Outdated
jeanas
jeanas previously approved these changes Apr 28, 2022

@jeanas jeanas left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je me permets une micro-remarque, autrement c'est bon pour moi. La nouvelle version est plus claire, ce qui est précieux pour les nouveaux. Merci beaucoup.

Comment thread CONTRIBUTING.rst Outdated
@PyDocTeur

Copy link
Copy Markdown

Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter.
Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: incorrect_title)
PyDocTeur v1.12.0

2 similar comments
@PyDocTeur

Copy link
Copy Markdown

Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter.
Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: incorrect_title)
PyDocTeur v1.12.0

@PyDocTeur

Copy link
Copy Markdown

Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter.
Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: incorrect_title)
PyDocTeur v1.12.0

@ghost ghost left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prise en compte des remarques.

@PyDocTeur

Copy link
Copy Markdown

Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter.
Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: incorrect_title)
PyDocTeur v1.12.0

@ghost

ghost commented Apr 28, 2022

Copy link
Copy Markdown
Author

Je ne peux actuellement pas faire de relecture propre à cause d'un problème posé par Sphinx avec l'url_licence.

@PyDocTeur

Copy link
Copy Markdown

Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter.
Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: incorrect_title)
PyDocTeur v1.12.0

@ghost

ghost commented Apr 28, 2022

Copy link
Copy Markdown
Author

De plus le PyDocTeur me persécute avec un titre de PR invalide (et pour cause) :-)
C'est quoi un label meta ?

Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter.
Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».

@PyDocTeur

Copy link
Copy Markdown

Hello @th71 ! Désolé, mais ton titre de pull request me semble invalide par rapport à ce que je suis programmé d'accepter.
Merci de le corriger ou d'ajouter le label meta si c'est une PR spéciale. Un exemple de titre valide serait : « Traduction de dossier/fichier.po ».


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: incorrect_title)
PyDocTeur v1.12.0

Validation des propositions (une seconde fois, je ne comprends pas pourquoi).

Co-authored-by: Jean Abou-Samra <jean@abou-samra.fr>
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>

@ghost ghost left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tout semble OK.

@jeanas

jeanas commented Apr 28, 2022

Copy link
Copy Markdown
Collaborator

C'est quoi un label meta ?

« meta » est un label défini pour ce dépôt GitHub. Les labels sont juste des « tags » qui se mettent sur les issues ou PR. Je le mets pour calmer PyDocTeur (il faut avoir les droits d'accès pour changer les labels).

@jeanas jeanas added the meta label Apr 28, 2022
@PyDocTeur

Copy link
Copy Markdown

ON Y EST PRESQUE ! Un p'tit label automerge et je merge ça !


Disclaimer

Je suis un robot fait par l'équipe de l'AFPy et de Traduction
sur leur temps libre. Je risque de dire des bétises. Ne me blâmez pas, blamez les développeurs.

Code source

I'm a bot made by the Translation and AFPy teams on their free
time. I might say or do dumb things sometimes. Don't blame me, blame the developer !

Source code

(state: approved)
PyDocTeur v1.12.0

@jeanas jeanas merged commit b0ab49b into python:3.10 Apr 28, 2022
@jeanas

jeanas commented Apr 28, 2022

Copy link
Copy Markdown
Collaborator

Tout est bon. Merci !

@ghost

ghost commented Apr 28, 2022

Copy link
Copy Markdown
Author

C'est moi, je n'arrivait pas à voir comment placer un label...
Encore des trucs a apprendre ;-)
Merci!

@ghost ghost deleted the CONTRIBUTING.rst branch May 8, 2022 08:36
JulienPalard pushed a commit that referenced this pull request May 22, 2022
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-authored-by: Jean Abou-Samra <jean@abou-samra.fr>
JulienPalard pushed a commit that referenced this pull request May 22, 2022
Co-authored-by: Christophe Nanteuil <35002064+christopheNan@users.noreply.github.com>
Co-authored-by: Jean Abou-Samra <jean@abou-samra.fr>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants