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

[betav18]corrige le MP de la migration #3491

Merged
merged 2 commits into from
Apr 6, 2016

Conversation

artragis
Copy link
Member

@artragis artragis commented Apr 3, 2016

Q R
Type de modification correction de bug
Ticket(s) (issue(s)) concerné(s) (#3490)

QA

Vérifiez juste l'orthographe et le fait que le message contient bien les infos.

@gustavi
Copy link
Contributor

gustavi commented Apr 4, 2016

@artragis
Copy link
Member Author

artragis commented Apr 4, 2016

lequel je dois changer?

@gustavi
Copy link
Contributor

gustavi commented Apr 4, 2016

1162 -> 1163

@artragis
Copy link
Member Author

artragis commented Apr 4, 2016

Voilà !

@artragis artragis changed the title corrige le MP de la migration [betav18]corrige le MP de la migration Apr 6, 2016
@artragis
Copy link
Member Author

artragis commented Apr 6, 2016

@gustavi ?

@gustavi
Copy link
Contributor

gustavi commented Apr 6, 2016

Parfait.

@gustavi
Copy link
Contributor

gustavi commented Apr 6, 2016

Juste un souci de PEP-8 et c'est good

@coveralls
Copy link

Coverage Status

Coverage increased (+16.4%) to 87.352% when pulling f01823a on artragis:patch-6 into 5d033a1 on zestedesavoir:release-v18.

@artragis
Copy link
Member Author

artragis commented Apr 6, 2016

done @sandhose @gustavi

@gustavi gustavi merged commit 8bca4c2 into zestedesavoir:release-v18 Apr 6, 2016
@artragis artragis deleted the patch-6 branch April 13, 2017 07:25
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