search

MTES-MCT / parcours-r

Valise pédagogique pour la formation à R
https://mtes-mct.github.io/parcours-r/
25 stars 14 forks source link

Modification M5 #40

Closed MurielleDelmotte closed 4 years ago

MurielleDelmotte commented 4 years ago

a verifier: la compilation du book ne voyant toujours pas les graphes/images

MaelTheuliere commented 4 years ago

Il manque ggspatial dans le fichier DESCRIPTION,c'est ce qui fait planter travis

RLesur commented 4 years ago

@MurielleL si je peux me permettre de te donner un petit conseil pour la prochaine pull request que tu feras (une fois que celle-ci sera fusionnée).
Lorsqu'on fait des pull requests, il est beaucoup plus pratique de faire ça depuis une branche de son propre fork.
L'idée c'est de conserver la branche master de son fork identique avec celle du repository auquel on contribue (git checkout master suivi de git pull upstream mastersuivi de git push origin master). Tu ne peux pas t'en douter mais tu vas certainement te retrouver coincée au moment où tu voudras faire ta seconde PR. Je trouve que c'est bien expliqué dans le livre de Jenny Bryan ici : https://happygitwithr.com/fork-and-clone.html#dont-touch-master Si ça peut te rassurer, tout le monde se fait avoir la 1ère fois, c'est pour ça que je me permets de te le signaler.

MaelTheuliere commented 4 years ago

Stéphane, je crois que tu as fermé cette pull request par inadvertance.

MaelTheuliere commented 4 years ago

Beau boulot @MurielleL ! 🥇