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

Ajout d'une section Bot #52

Closed
wants to merge 1 commit into from
Closed

Conversation

DeltaEvo
Copy link
Contributor

@DeltaEvo DeltaEvo commented Jun 7, 2018

Je demande l'ajout de mon bot MaidChan pour pouvoir profiter d'une de ces fonctionalité, l'evaluation de code

(Pour le Javascript elle utilise RunKit donc on n'a accès aux packages npm)

Mais elle ne se limite pas qu'a ça, Maid Chan peux vous rapeller les status code HTTP (Avec une petite image de chien, de chat, ou de wc)

Pour le moment elle supporte: Recherche de CommitStrip + XKCD, Encodage Decodage de Base64

@DeltaEvo
Copy link
Contributor Author

DeltaEvo commented Jun 9, 2018

Je precise que l'on peut ajouter un bot sur Discord en lui donnant des permissions particulière, ainsi il n'est pas administrateur

@Xstoudi Xstoudi requested review from Xstoudi, RomainLanz, nolanrigo, fraxken, Purexo, antoineneff, Westixy and tiyodev and removed request for Westixy June 9, 2018 16:46
Copy link
Collaborator

@Purexo Purexo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne suis pas pour l'ajout du bot Maid Chan.
Je ne penses pas que la multiplication soit une bonne chose et que ce sois pertinent dans notre communauté.

Si on souhaite des fonctionnalités supplémentaires, je penses qu'elles doivent être ajouté au bot existant (et bien évidement être review par la commu).

Par contre je suis pour l'ajout de la section bot tout en précisant que à chaque demande, une PR/issue.

@DeltaEvo
Copy link
Contributor Author

@Purexo, @Xstoudi m'avais deja proposé ça, mais je doit t'avouer que ça m'arrange pas trop, car actuellement le système pour l'evaluation de code est celui de runkit. Runkit ne propose pas d'api officielle donc dans MaidChan j'ai un client issue du Reverse Engineering de leur système ceci pose quelques problèmes de stabilité et donc fait que je doit parfois corriger des poblèmes car pour que ça fonctionne je doit avoir une AST conforme à celle du client RunKit, ... un vrai bordel https://github.com/DeltaEvo/MaidChan/blob/master/src/modules/Eval/runkit.js

Ce bot n'as pas été fait à l'origine pour ES Community, je me suis juste rendu compte qu'il serait utile la bas aussi, donc si je devait l'integrer à es_bot je peux:

  • Dupliquer le code: Ça ne m'arrange pas trop pour des raisons evidantes
  • Le publier sur npm: J'ai pas trop envie de jouer avec le feu et de publier un client runkit et je devrait quand même mettre à jour les dependances sur les deux bots

Voila j'hespère que tu comprend mes raisons

@Xstoudi
Copy link
Member

Xstoudi commented Jun 11, 2018

Cela dit tu pourrais aussi faire que l'ES-Bot choppe le package depuis un dépôt GitHub et non pas depuis le dépôt NPM, non ?

@DeltaEvo
Copy link
Contributor Author

Ouep je peux faire ça s'est vrai 🤔
Mais ceci necessitera quand même la separation de mon fichier runkit et de mettre à jour les deux bots
Mais c'est une solution plus viable que les deux que j'ai ennoncées c'est vrai
Si jamais ça vous derange vraiment d'avoir un autre bot dont vous ne controllez pas les features sur l'ES Community on peux essayer de partir la dessus, ça va me demander un peu plus de boulot mais c'est pas la mort 😄

@Xstoudi
Copy link
Member

Xstoudi commented Jun 11, 2018

Yes effectivement tu dois séparer, mais ça reste un bon compromis entre ne pas push sur NPM et ne pas rajouter un bot sur le Discord je pense.

Si t'as du time pour faire ça, fais-toi plaisir.

@DeltaEvo
Copy link
Contributor Author

Uep, je vais quand même voir si y'a moyen de m'eviter cette separation, mais si jamais comme dis ci dessus avoir un bot dont vous ne controllez pas les features est derangeant je partirait la dessus (Bien que je préférerais ne rien separer xD)

@Purexo
Copy link
Collaborator

Purexo commented Jun 11, 2018

Je penses que ce sera le plus pertinent. (et qui sais, peut être que d'autres tomberont sur ta lib et serons heureux de l'utiliser ^^)

@DeltaEvo
Copy link
Contributor Author

@Purexo j'ai pas trop envie que ma lib se fasse remarquer xD, mais moui

@Xstoudi
Copy link
Member

Xstoudi commented Aug 3, 2018

T'en es où @DeltaEvo ?

@DeltaEvo
Copy link
Contributor Author

DeltaEvo commented Aug 3, 2018

J'ai toujours rien isolé, il faut que je m'en occupe 😄

@Xstoudi
Copy link
Member

Xstoudi commented Aug 3, 2018

Pas de souci, donne-nous des nouvelles :)

@fraxken
Copy link
Member

fraxken commented Aug 21, 2018

Je suppose que cet demande doit être transférer vers es_bot du coup non ?

@Xstoudi
Copy link
Member

Xstoudi commented Aug 21, 2018

En l'occurence on traite deux demandes :

  • Ajout de la fonctionnalité (à mettre dans es_bot)
  • Ajout du channel "bot" (à mettre dans le CoC)

@DeltaEvo DeltaEvo changed the title Ajout d'une section Bot et demande d'ajout du bot Maid Chan Ajout d'une section Bot Aug 24, 2018
@DeltaEvo
Copy link
Contributor Author

J'ai separé les deux demandes pour pouvoir traiter cette Pull Request

@fraxken
Copy link
Member

fraxken commented Nov 6, 2019

Je ferme la PR (je pense que c'est maintenant un peu hors date).

@fraxken fraxken closed this Nov 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants