Skip to content

Conversation

@saschasalles
Copy link
Contributor

Fix #1085
First pull request ever!

@Seluj78 Seluj78 mentioned this pull request Dec 13, 2019
@Seluj78 Seluj78 added the NEED REVIEWER This PR needs a reviewer :) label Dec 13, 2019
@awecx awecx removed the NEED REVIEWER This PR needs a reviewer :) label Dec 13, 2019
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.
Je t'ai mis quelques remarques à la fin.

saschasalles and others added 3 commits December 15, 2019 11:56
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
Co-Authored-By: Antoine <43954001+awecx@users.noreply.github.com>
@vpoulailleau
Copy link
Contributor

@Sascha40 Le test en automatique dit qu'au moins une des lignes de ton fichier est maintenant trop longue (padpo doit aussi te le dire si tu le repasses). Tu peux tenter pour corriger cela :

source venv/bin/activate
powrap reference/expressions.po

Tu peux aussi le corriger à la main si la correction est facile.

Puis committer et pusher le résultat (n'oublie pas le pull avant tout ça, cf les diapos vues pendant l'atelier)

@awecx
Copy link

awecx commented Dec 17, 2019

Ping :-)

@Seluj78
Copy link
Collaborator

Seluj78 commented Feb 4, 2020

Hello @Sascha40, ping !

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.

Je travaille sur reference/expressions.po

4 participants