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.

Sign up for GitHub

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

Jump to bottom

[Py3k]: utilise __str__ #4037

Merged
merged 3 commits into from Dec 8, 2016
Merged

[Py3k]: utilise __str__ #4037

merged 3 commits into from Dec 8, 2016

Conversation

artragis
Copy link
Member

@artragis artragis commented Dec 5, 2016

Q R
Type de modification évolution
Ticket(s) (issue(s)) concerné(s) (ex #4013)

NO QA

@coveralls
Copy link

coveralls commented Dec 7, 2016 •
edited

Coverage Status

Coverage decreased (-0.06%) to 87.617% when pulling 122c99d on artragis:fix_4013 into c69a7db on zestedesavoir:dev.

@pierre-24
Copy link
Member

Je viens de checker avec pycharm, à priori il en manque pas.

@pierre-24 pierre-24 merged commit 5bc1357 into zestedesavoir:dev Dec 8, 2016
@pierre-24 pierre-24 added C-Back Concerne le back-end Django S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité labels Dec 8, 2016
@pierre-24 pierre-24 added this to the Version de développement milestone Dec 8, 2016
Emeric54 pushed a commit to Emeric54/zds-site that referenced this pull request Dec 19, 2016
@artragis @Emeric54
[Py3k]: utilise __str__ (zestedesavoir#4037)
41bee6b 
* prépare les modèles à python3 : les méthodes __str__

* pep8

* bad merging
@pierre-24 pierre-24 mentioned this pull request Dec 26, 2016
Closed
@artragis artragis deleted the fix_4013 branch April 13, 2017 07:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
C-Back Concerne le back-end Django S-Refactorisation Améliore le code existant sans forcément ajouter de nouvelle fonctionnalité
Projects
None yet
Milestone
v22
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@artragis @coveralls @pierre-24