afup / aperophp

Nouvelle version du site aperophp.net
aperophp.net
21 stars 16 forks source link

Correction du fichier après l'issue 122 #147

Closed rkueny closed 8 years ago

rkueny commented 9 years ago

J'ai essayé d'installer le projet en local et j'avais l'erreur suivante lors du ./bin/assets.sh

ERROR: Can't find config file: aperophp/vendor/twitter/bootstrap/js/.jshintrc

J'ai donc corrigé le fichier assets.sh en remplaçant twitter par twbs. Changement fait ici : https://github.com/afup/aperophp/pull/122

rkueny commented 9 years ago

Et j'ai encore des erreurs après au final :

aperophp/vendor/twbs/bootstrap/js/bootstrap-alert.js: line 23, col 17, Bad option: ';_;'. aperophp/vendor/twbs/bootstrap/js/bootstrap-button.js: line 23, col 17, Bad option: ';_;'.

Une idée de comment régler ça ?

mikaelrandy commented 9 years ago

Comment obtiens tu ces nouvelles erreurs ?

mikaelrandy commented 9 years ago

C'est bon, j'ai trouvé : j'avais une vieille version de jshint.

Du coup, c'est ça le soucis : https://github.com/jshint/jshint/issues/395 Maintenant, jshint affiche ces erreurs qui ont toujours eu lieu, mais qui n'étaient pas affichées.

mikaelrandy commented 9 years ago

J'ai repris la PR, et je l'ai terminée : #148