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

Shipdict ref #46

Merged
merged 4 commits into from
Dec 12, 2020
Merged

Shipdict ref #46

merged 4 commits into from
Dec 12, 2020

Conversation

aollier
Copy link
Contributor

@aollier aollier commented Dec 6, 2020

J'ai fait trois modifications :

  1. fix Exercice ShipDict : les fichiers de référence contiennent des données répliquées #44
  2. la méthode is_abbreviated peut être déclarée comme statique puisqu'elle est indépendante de l'instance
  3. J'ai mis à jour les lignes du fichier qui compare la taille des fichiers texte "normal" et bavard. Aussi, j'ai enlevé le fichier v2.0 vide de la ligne et de l'archive zip dans laquelle il était.

Adrien

is_abbreviated est indépendant de l'instance
les fichiers de référence sont plus petits
après qu'on a enlevé les répliques

le fichier v2.0 a été supprimé
data/shipdict.py Show resolved Hide resolved
data/shipdict.py Outdated Show resolved Hide resolved
data/shipdict.py Show resolved Hide resolved
@parmentelat
Copy link
Contributor

parmentelat commented Dec 11, 2020

ça me plait bien

j'ai fait deux-trois micro-remarques dans le code
si tu as le temps à l'occasion

@parmentelat parmentelat merged commit d8f7e00 into flotpython:master Dec 12, 2020
@parmentelat
Copy link
Contributor

merci Adrien

@aollier aollier deleted the shipdict-ref branch December 12, 2020 17:15
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.

Exercice ShipDict : les fichiers de référence contiennent des données répliquées
2 participants