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

La branche dev est cassée #2241

Closed
GerardPaligot opened this issue Feb 7, 2015 · 4 comments
Closed

La branche dev est cassée #2241

GerardPaligot opened this issue Feb 7, 2015 · 4 comments
Assignees
@SpaceFox
Labels
S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Milestone
Version 1.6

Comments

@GerardPaligot
Copy link
Member

Nous avons la branche dev à nouveau cassée. Tous les tests passent mais, après ça, l'erreur suivante est afficher dans Travis :

back_mysql runtests: commands[1] | flake8
./zds/utils/templatetags/emarkdown.py:33:37: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:33:39: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:35:34: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:35:36: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:37:45: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:37:47: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:42:38: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:42:40: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:45:41: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:45:43: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:48:42: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:48:44: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:50:35: E251 unexpected spaces around keyword / parameter equals
./zds/utils/templatetags/emarkdown.py:50:37: E251 unexpected spaces around keyword / parameter equals
ERROR: InvocationError: '/home/travis/build/zestedesavoir/zds-site/.tox/back_mysql/bin/flake8'
___________________________________ summary ____________________________________
ERROR:   back_mysql: commands failed
The command "tox $TEST_APP" exited with 1.

Quelqu'un comprend le problème ?
@SpaceFox SpaceFox mentioned this issue Feb 7, 2015
Closed
@poulp
Copy link
Contributor

poulp commented Feb 7, 2015

Logiquement il faut enlever les espaces entre les = dans les arguments de la fonctions appelée, mais ce que je comprend pas c'est que flake8 ne m'alerte pas en local

@Situphen
Copy link
Member

Situphen commented Feb 7, 2015

ce que je comprend pas c'est que flake8 ne m'alerte pas en local

tu utilises bien la commande tox -e flake8 ?

@gustavi gustavi self-assigned this Feb 7, 2015
@SpaceFox SpaceFox added Bloquant Ticket qui doit être traité avant la prochaine mise à jour S-Régression Corrige un problème sur un composant qui fonctionnait auparavant labels Feb 7, 2015
@SpaceFox SpaceFox modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 7, 2015
@SpaceFox SpaceFox mentioned this issue Feb 7, 2015
Closed
@SpaceFox SpaceFox mentioned this issue Feb 7, 2015
Merged
@SpaceFox SpaceFox assigned SpaceFox and unassigned gustavi Feb 7, 2015
SpaceFox added a commit that referenced this issue Feb 7, 2015
@SpaceFox
Merge pull request #2248 from SpaceFox/fix_2241
70db0ca 
Fixes #2241: Devrait empêcher Travis/tox/flake8 de râler
@SpaceFox SpaceFox removed the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Feb 7, 2015
@SpaceFox SpaceFox modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 7, 2015
@SpaceFox
Copy link
Contributor

SpaceFox commented Feb 7, 2015

C'est corrigé, mais je ne sais toujours pas pourquoi du code vieux de presque 1 an pète seulement maintenant...

@GerardPaligot
Copy link
Member Author

Technique ninja ça. Attendre pour devenir invisible et attaquer quand on s'y attend le moins.

@gustavi gustavi mentioned this issue Feb 8, 2015
Merged
Situphen pushed a commit to Situphen/zds-site that referenced this issue Feb 13, 2015
@SpaceFox @Situphen
Fixes zestedesavoir#2241: Devrait empêcher Travis/tox/flake8 de râler
9e38c0f
Situphen pushed a commit to Situphen/zds-site that referenced this issue Feb 13, 2015
@Situphen
Fix zestedesavoir#2241 : corrections de PEP8 sur le templatetag « ema…
f9a9dfa 
…rkdown »
Situphen pushed a commit to Situphen/zds-site that referenced this issue Feb 13, 2015
@Situphen
Fix zestedesavoir#2241 : corrections de PEP8 sur tous le projet
58fd97f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SpaceFox SpaceFox

Labels
S-Régression Corrige un problème sur un composant qui fonctionnait auparavant
Projects
None yet
Milestone
Version 1.6
Development

No branches or pull requests
5 participants
@poulp @GerardPaligot @gustavi @SpaceFox @Situphen