Code-of-conduct icon indicating copy to clipboard operation
Code-of-conduct copied to clipboard

Coquilles dans les blocs de code

Open Aakodal opened this issue 5 years ago • 1 comments

J'ai remarqué certaines "coquilles" dans les blocs de code qui sont surtout de l'ordre de la consistance du coding style :

  • Sur les 4 strings non template strings dans les deux premiers blocs, vous alternez single quotes et double quotes
  • Dans le bloc sur le code de conduite, vous avez des espaces en trop après la parenthèse du join et avant la virgule des arguments
  • Dans le bloc sur les invitations, il manque l'espace avant la virgule dans les arguments du addEventListener. Par ailleurs, mais ça c'est discutable, la forme function(arg) {} pour un callback me semble un peu désuète depuis ES2015 et les arrow functions
  • Dans le bloc sur les salons, le console.log, en plus de manquer le point-virgule final présent partout ailleurs, n'est pas correct (cf #102)

P-S : je ne fais pas de PR étant donné que pour certaines "coquilles", ça dépend du coding style voulu de base, je ne vais donc pas imposer le mien.

Aakodal avatar Nov 28 '20 21:11 Aakodal

Je suis d'accord. J'avais aussi remarquer ces "coquilles". Je propose :

const you = ESCommunity.currentUser();

if (!you.hasReadCodeOfConduct) {
  throw new Error('Merci de lire le code de conduite !');
}
you.setChannel('Presentation');
you.write('... Hello world !');
fs.createReadStream(path.join(__dirname, 'code_of_conduct.txt')).pipe(process.stdout);
for await (const line of ESCommunity.prohibedBehaviors()) {
  console.log(`- ${line}`);
}
document.getElementById('discordInvitation').addEventListener('click', (e) => {
  e.preventDefault(); 
  ESCommunity.invite(user);
});
const channels = ESCommunity.getChannels();
for (const [name, description] of channels) {
  console.log(`- \`#${name}\` - ${description}`);
}

Mesteery avatar Nov 28 '20 21:11 Mesteery

Bonjour,

Je clos cette issue qui est très vielle, il me semble que le CoC à subit de grosse révisions depuis. Au besoin il ne faut vraiment pas hésiter à faire des PR, le CoC est un document collaboratif, conçu pour et par la communauté.

Si il y a des sujets à débats comme le coding style, ils pourront être débattu dans la PR

Purexo avatar Sep 13 '22 14:09 Purexo