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

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#75,596 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
72/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
eslint, javascript, node.js
Bereich
testing, tooling

Rechercherichtung

Beginne mit eslint-factory/eslint.config.cjs und prüfe, wie dessen Ignorierregeln auf actions/setup/js angewendet werden. Vergleiche die vorhandenen Dateien in diesem Verzeichnis mit dem vorgeschlagenen Ausschluss und führe anschließend make lint:setup-js aus, um zu bestätigen, dass die vendorte Datei keine gh-aw-custom-Diagnosen verursacht. Erledigt ist die Aufgabe, wenn der Ausschluss die vendorte Datei abdeckt, ohne von Teammitgliedern erstellte Dateien auszuschließen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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
Vorherrschende Sprache
Go
Sterne
5.6k
Forks
615
Ø Merge
5 Std. 44 Min.
Gemergte PRs (30 T.)
808

Entwicklungsumgebung

In Codespaces öffnen

Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus github/gh-aw

Alle Issues in github/gh-aw

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.