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

Corrige une erreur dans Gulpfile.js #2418

Merged
merged 1 commit into from Mar 11, 2015

Conversation

@pierre-24
Copy link
Member

commented Mar 11, 2015

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

Parce qu'il faut toujours que les erreurs de gulp me tombent dessus ...

Note de QA

Attention, nécéssite les outils du front

  • Aller sur la branche upstream/dev, et faire npm run gulp -- build, constater que certaines images ne fonctionnent plus (en fait sprite.png n'est plus chargé, vous pouvez vérifier ça dans la console de votre navigateur préféré).
  • Aller sur ma branche et refaire de même. Constater que ça fonctionne.

Ping @Situphen sur celle là.

@landscape-bot

This comment has been minimized.

Copy link

commented Mar 11, 2015

Code Health
Repository health decreased by 0.11% when pulling 36de56b on pierre-24:fix_gulpfile into 732bb29 on zestedesavoir:dev.

Gulpfile.js Outdated
@@ -165,7 +165,7 @@ gulp.task("sprite", function() {
.pipe(sprite({
name: "sprite",
style: "_sprite.scss",
cssPath: "../images",
cssPath: "../images/",

This comment has been minimized.

Copy link
@Situphen

Situphen Mar 11, 2015

Contributor

Peux-tu carrément mettre ce code s'il te plait ? :)

      cssPath: "../" + imagesDir,

This comment has been minimized.

Copy link
@pierre-24

pierre-24 Mar 11, 2015

Author Member

Fait :) (et ça marche toujours)

@pierre-24 pierre-24 force-pushed the pierre-24:fix_gulpfile branch from 36de56b to 27d7a98 Mar 11, 2015

@pierre-24 pierre-24 force-pushed the pierre-24:fix_gulpfile branch from 27d7a98 to 9340f89 Mar 11, 2015

@Situphen

This comment has been minimized.

Copy link
Contributor

commented Mar 11, 2015

Tu as eu ce problème car tu as réinstallé les dépendances, j'ai testé et je l'ai eu aussi. :)

Rapport de QA : Parfait, à merger !

@landscape-bot

This comment has been minimized.

Copy link

commented Mar 11, 2015

Code Health
Repository health decreased by 0.11% when pulling 9340f89 on pierre-24:fix_gulpfile into 732bb29 on zestedesavoir:dev.

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Mar 11, 2015

Juste avant de merger : c'était quoi les répercussions de ne pas embarquer le fichier sprite.png ?

@Situphen

This comment has been minimized.

Copy link
Contributor

commented Mar 11, 2015

Je ne comprend pas ce que tu veux dire par "embarquer". Le fichier " sprite.png" contient toutes les icônes du site.

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Mar 11, 2015

Ce que j'aimerais savoir, c'est comment ça se fait que je vois des icones chez moi avec des branches dérivées de la branche dev et si le problème est déjà en prod.

@Situphen

This comment has been minimized.

Copy link
Contributor

commented Mar 11, 2015

En fait, il n'y a pas de problème tant que tu ne réinstalle pas les dépendances npm de zds. Ce qui veut dire que chez toi, sur la prof et chez d'autres ça fonctionne mais que quand il mettront à jour leur dépendances, ça ne fonctionnera plus ! ;)

@GerardPaligot

This comment has been minimized.

Copy link
Member

commented Mar 11, 2015

Ah ok, merci pour les précisions.

GerardPaligot added a commit that referenced this pull request Mar 11, 2015
Merge pull request #2418 from pierre-24/fix_gulpfile
Corrige une erreur dans Gulpfile.js

@GerardPaligot GerardPaligot merged commit 35c176b into zestedesavoir:dev Mar 11, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@GerardPaligot GerardPaligot added this to the Version 1.7 milestone Mar 11, 2015

@pierre-24 pierre-24 deleted the pierre-24:fix_gulpfile branch Apr 16, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.