Skip to content

venv.po, unicode.po, datastructures.po complete #917

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

Merged
merged 27 commits into from
Dec 29, 2019
Merged

Conversation

drNymbus
Copy link
Contributor

No description provided.

Copy link
Collaborator

@Seluj78 Seluj78 left a comment

Choose a reason for hiding this comment

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

Merci pour cette review ! 🎉

drNymbus and others added 4 commits November 1, 2019 13:51
Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
Co-Authored-By: Jules Lasne (jlasne) <[email protected]>
Copy link
Contributor

@christopheNan christopheNan left a comment

Choose a reason for hiding this comment

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

Quelques petits points et c'est bon.

Copy link

@awecx awecx left a comment

Choose a reason for hiding this comment

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

Merci pour cette PR.
Pour tes futures contributions, n'hésite pas à faire une PR par fichier (même pour quelques lignes).
C'est plus lisible est cela permet de merger plus rapidement.

@deronnax
Copy link
Collaborator

deronnax commented Nov 5, 2019

Merci pour cette PR.
Pour tes futures contributions, n'hésite pas à faire une par fichier (même pour quelques lignes).
C'est plus lisible est cela permet de merger plus rapidement.

(c'est un peu ma faute, je lui ai dit trop tard de faire des PR séparées)

drNymbus and others added 8 commits November 7, 2019 14:09
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
Co-Authored-By: Christophe Nanteuil <[email protected]>
@Seluj78 Seluj78 added NEED REVIEWER This PR needs a reviewer :) awaiting changes and removed awaiting changes NEED REVIEWER This PR needs a reviewer :) labels Nov 15, 2019
@Seluj78
Copy link
Collaborator

Seluj78 commented Nov 17, 2019

@drNymbus il faut que tu résoude les conflits dans github ou alors que tu ailles dans ta cmdline sur la bonne branche, mets a jour ton upstream avec git fetch upstream puis git rebase upstream/3.8, résouds les conflits puis git push --force

@Seluj78 Seluj78 self-assigned this Dec 11, 2019
@Seluj78 Seluj78 self-requested a review December 11, 2019 15:02
@Seluj78
Copy link
Collaborator

Seluj78 commented Dec 27, 2019

ping @drNymbus 🏓

@drNymbus
Copy link
Contributor Author

Merci infiniment de votre patience ! Du coup je viens de régler les conflits, il me reste seulement a passer le powrap, il me dit que certains espaces doivent êtres insecables, j'ai alors rajouté ces espaces mais rien ne se passe... une idée ?

@awecx
Copy link

awecx commented Dec 27, 2019

Merci infiniment de votre patience ! Du coup je viens de régler les conflits, il me reste seulement a passer le powrap, il me dit que certains espaces doivent êtres insecables, j'ai alors rajouté ces espaces mais rien ne se passe... une idée ?

Ce n'est pas powrap qui te dit ça, mais padpo, un outil encore en béta qui donne des indications supplémentaires.

Powrap te demande seulement de réindenter expressions.po (l.333 de la sortie Travis).

@Seluj78
Copy link
Collaborator

Seluj78 commented Dec 28, 2019

@drNymbus -> @vmelancon same same ?

Copy link
Collaborator

@Seluj78 Seluj78 left a comment

Choose a reason for hiding this comment

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

All good à part les modifications sur les espaces insécables sur la doc originale

Copy link

@awecx awecx left a comment

Choose a reason for hiding this comment

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

Encore une modification et ça sera bon.

Copy link

@awecx awecx left a comment

Choose a reason for hiding this comment

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

:1:

@awecx awecx dismissed stale reviews from Seluj78 and christopheNan December 29, 2019 18:02

Modifications demandées corrigées.

@awecx awecx merged commit 4a74866 into python:3.8 Dec 29, 2019
@awecx
Copy link

awecx commented Dec 29, 2019

Bon travail @drNymbus !
N'hésite pas à reprendre un fichier si tu le souhaites 😃

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.

5 participants