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

Bump des dépendances npm + clean du Gulpfile + fix les warnings JSHint #2571

Merged
merged 4 commits into from
Apr 23, 2015

Conversation

sandhose
Copy link
Contributor

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? non
Tickets (issues) concernés --

Y'a pas mal de petites choses dans cette PR qui touchent toutes au système de build du front.

  • update des dépendances npm aux dernières versions
  • fixe la version de css-sprite pour éviter qu'il casse tout (ce paquet utilise des binaries, et a tendance à pas être super stable...)
  • rebuild le css des pages d'erreurs ; permet d'avoir normalize.css + autoprefixer à jour
  • clean le gulpfile de trucs inutiles (browserify + des options qui servent à rien dans la config)
  • corrige les warnings JSHint

QA

Mettre à jour les dépendances, et virer les dépendances inutilisées (npm install && npm prune)
Clean + build le front (npm run clean && npm run build)
Vérifier que le front est bien généré (donc tester globalement le site)

 - passe tous les chemins "composés" par path.join
 - vire browserify qui ne sert pas encore (et qui servira peut-être
   jamais)
 - corrige quelques erreurs de config (options inutiles, mauvais chemins
   pour le scss des erreurs)
@sandhose sandhose added the C-Front Concerne l'interface du site label Apr 18, 2015
@coveralls
Copy link

Coverage Status

Coverage remained the same at 79.53% when pulling 62b2880 on sandhose:npm-deps-bump into 4fa0c03 on zestedesavoir:dev.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling 62b2880 on sandhose:npm-deps-bump into 4fa0c03 on zestedesavoir:dev.

@DevHugo
Copy link
Contributor

DevHugo commented Apr 20, 2015

J'ai réussi à compiler le front (sous unix), je ne me prononce pas sur le code, je ne connaît pas les outils.

@Situphen
Copy link
Member

Pour que tout le monde ait les mêmes versions des dépendances, peux-tu les mettre toutes en strictes ? :)

@sandhose
Copy link
Contributor Author

Pour que tout le monde ait les mêmes versions des dépendances, peux-tu les mettre toutes en strictes ? :)

Done !

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling df9d842 on sandhose:npm-deps-bump into 4fa0c03 on zestedesavoir:release-v1.8.

@pierre-24
Copy link
Member

Ça marche toujours donc je merge :)

pierre-24 added a commit that referenced this pull request Apr 23, 2015
Bump des dépendances npm + clean du Gulpfile + fix les warnings JSHint
@pierre-24 pierre-24 merged commit 99280c7 into zestedesavoir:dev Apr 23, 2015
@pierre-24 pierre-24 added this to the Version 1.9 milestone Apr 23, 2015
@SpaceFox SpaceFox modified the milestones: Version 15.5.1, Version de développement May 26, 2015
@sandhose sandhose deleted the npm-deps-bump branch January 6, 2016 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants