eslint-factory: no ignores entry for vendored actions/setup/js/work_queue_yaml.cjs ([email protected] copy) causes unactionable lint no

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#75,596 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

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

Piste de recherche

Commencez par eslint-factory/eslint.config.cjs et examinez comment ses exclusions s’appliquent à actions/setup/js. Vérifiez les fichiers existants dans ce répertoire par rapport à l’exclusion proposée, puis exécutez make lint:setup-js pour confirmer que le fichier tiers intégré ne produit aucun diagnostic gh-aw-custom. La tâche est terminée lorsque l’exclusion couvre le fichier tiers intégré sans exclure les fichiers créés par l’équipe.

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

Description

cookie eslint
Summary

actions/setup/js/work_queue_yaml.cjs opens with:

// @ts-nocheck
// Generated by scripts/work_queue_yaml/build.mjs from [email protected].

It is a byte-for-byte vendored copy of the yaml npm package, not team-authored code. eslint-factory/eslint.config.cjs's only ignores entry is ["**/*.test.cjs", "**/*.test.js"] (confirmed by direct read) — there is no exclusion for vendored/generated files, and the lint:setup-js Makefile target runs ESLint (all ~70 gh-aw-custom/* rules) against the whole actions/setup/js directory with this config, so every rule lints this file.

Live grounding

Grepped the full non-test corpus for a @ts-nocheck + Generated by vendoring marker: work_queue_yaml.cjs is currently the only file carrying this convention. It alone accounts for 13 live .substr() call sites (no-deprecated-string-substr would warn on all of them — re-confirmed today, line numbers: 998, 1025 ×2, 1026, 2044, 2248, 2420, 3635, 3689, 3694, 4325, 4351, 4672, 4746), plus whatever other gh-aw-custom/* rules independently fire on the same 4700+-line file. None of this is actionable: fixing a vendored upstream copy would just be overwritten by the next build.mjs regeneration from [email protected], and the file is explicitly marked @ts-nocheck for exactly this reason.

Suggested fix

Add an ignores entry to eslint-factory/eslint.config.cjs scoped to vendored/generated files — either an explicit path ("**/work_queue_yaml.cjs") or, more durably, a convention-based exclusion that detects the // Generated by + // @ts-nocheck marker pattern so future vendored files are automatically excluded without a config change each time.

Acceptance criteria
  • actions/setup/js/work_queue_yaml.cjs (or any file matching the vendored-file convention) is excluded from gh-aw-custom/* linting via eslint.config.cjs.
  • make lint:setup-js (or equivalent) produces zero gh-aw-custom/* diagnostics for this file after the change.
  • The exclusion does not widen to cover genuinely team-authored files — verify no other actions/setup/js/*.cjs file currently matches the chosen ignore pattern.

Generated by 🤖 ESLint Refiner · claude · agent · ~122 AIC · ⌖ ~1 AIC · ⊞ 4.2K · ◷
Claim finished

  • expires on Nov 13, 2026, 7:34 PM UTC-08:00
Langage dominant
Go
Étoiles
5.6k
Forks
615
Merge moyen
5 h 44 min
PR mergées (30 j)
808

Préparer son environnement

Ouvrir dans Codespaces

Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.

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 github/gh-aw

Toutes les issues de github/gh-aw

Issues similaires

Plus d'issues Go

Recevez les nouvelles issues par e-mail

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