addon-verify: re-indents the test.js it generates

Ouverte Adaptée aux débutants
#1,199 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 2 jours

@Mittalnew y travaille déjà.

Depuis le 30/10/2026.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
84/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
javascript, node.js
Domaine
tooling

Piste de recherche

Dans utils/generateFileList.mjs, commencez par lire updateJsRequirePaths et la façon dont cette fonction utilise dedent pour créer test.js. Conservez l’en-tête généré tout en préservant l’indentation de l’extrait telle qu’elle apparaît dans l’issue. Le travail est terminé lorsque le test.js généré conserve cette mise en forme ; l’issue ne précise aucun test à exécuter.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

good first issue help wanted

updateJsRequirePaths in utils/generateFileList.mjs builds test.js inside a dedent template, so dedent also strips indentation from the snippet it wraps. For this test.js block:

const addon = require('./build/Release/addon');

for (const name of ['a', 'b']) {
  console.log(addon.hello(name));
}

it writes:

'use strict';
  const common = require('../../common');
  const addon = require(`./build/${common.buildType}/addon`);

for (const name of ['a', 'b']) {
console.log(addon.hello(name));
}

The test still runs, it's only how the file looks. Prepending the two header lines as a plain string instead of going through dedent would keep the snippet as written.

Langage dominant
JavaScript
Étoiles
70
Forks
79
Merge moyen
1 j 21 h
PR mergées (30 j)
39

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de nodejs/doc-kit

Toutes les issues de nodejs/doc-kit

Issues similaires

Plus d'issues JavaScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.