Skip to content

Charles Demers
charlesdemers

Organizations

@mirego
@charlesdemers
Fix `hashFn` reference
1 commit with 3 additions and 1 deletion
@charlesdemers
@charlesdemers
charlesdemers commented on pull request mirego/st-js-gradle-plugin#17
@charlesdemers

Mmmm c'est embêtant parce que je ne l'ai pas commité, c'est un truc que j'ai fait sur mon poste et j'attendais pour merger...

charlesdemers commented on pull request mirego/st-js-gradle-plugin#17
@charlesdemers

Ouais c'est parfait, je voulais juste dire que ton template ne devrait pas utiliser de fichier .babelrc :wink:

charlesdemers commented on pull request mirego/st-js-gradle-plugin#17
@charlesdemers

Il faudrait faire comme dans scratch et mettre la config de Babel dans le package.json pour éviter des clashes si jamais on nest des dossiers qui d…

charlesdemers commented on pull request mirego/Kalendae#3
@charlesdemers

:+1:

charlesdemers commented on pull request mirego/Kalendae#3
@charlesdemers

charlesdemers commented on pull request mirego/st-js#78
@charlesdemers

lgtm :+1:

charlesdemers commented on pull request mirego/st-js#77
@charlesdemers

Après un gros squash :+1:

charlesdemers commented on pull request mirego/st-js#74
@charlesdemers

Nice! :+1:

charlesdemers commented on pull request mirego/st-js#66
@charlesdemers

Cool :+1:

@charlesdemers

replaceAll n’existe pas, si tu veux remplacer toutes les occurrences tu dois passer par une RegExp: var targetRegExp = new RegExp(target, 'g'); // …

charlesdemers commented on pull request mirego/st-js#65
@charlesdemers

L'iterator que tu reçois ici ne sert à rien? C'est quoi le use-case en Java?

charlesdemers commented on pull request mirego/st-js#63
@charlesdemers

Ah! get it! Carry on :smile:

charlesdemers commented on pull request mirego/st-js#63
@charlesdemers

Est-ce que c'est utile d'avoir un objet plutôt que la primitive true?

charlesdemers commented on pull request mirego/st-js#61
@charlesdemers

Étant donné que tu call la méthode avec exactement les mêmes paramètres tu pourrais faire: String.prototype.indexOf$String = String.prototype.indexOf…

charlesdemers commented on pull request mirego/st-js#61
@charlesdemers

this.length n’est pas nécéssaire ici, c'est ça par défaut si tu ne passe pas le 2ème paramètre.

charlesdemers commented on pull request mirego/st-js#55
@charlesdemers

Ah! I get it now! merci :)

charlesdemers commented on pull request mirego/st-js#55
@charlesdemers

C'est normal que cette méthode ait exactement le même comportement que celle d'au-dessus?

charlesdemers commented on pull request mirego/mirego-open-web#20
@charlesdemers

Woot! :+1:

charlesdemers commented on pull request mirego/st-js#35
@charlesdemers

:+1:

charlesdemers commented on pull request mirego/st-js#28
@charlesdemers

Pourrais-tu mettre un exemple ce que ça génère comme code?

charlesdemers commented on pull request mirego/st-js-gradle-plugin#2
@charlesdemers

Ça serait bien d'enlever ça complètement :)

charlesdemers commented on pull request mirego/st-js#18
@charlesdemers

Il fait ça mais il check valueOf avant, et comme valueOf sur un enum n'a pas la même sémantique, ça risque de péter! Ref: https://github.com/mirego…

charlesdemers deleted branch feature/update-linter-file at mirego/javascript-style-guide
@charlesdemers
@charlesdemers
Update linter file
1 commit with 73 additions and 85 deletions
@charlesdemers
Update linter file
1 commit with 73 additions and 85 deletions
Something went wrong with that request. Please try again.