`createLockFileForStandardQuery` always creates temp lock because `packPacklist` doesn't return lock files

Offen Anfängerfreundlich
#4,471 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
68/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Ruhig
Tech-Stack
typescript, vscode
Bereich
devtools

Rechercherichtung

Beginne in src/local-queries/standard-queries.ts bei createLockFileForStandardQuery und verfolge das Ergebnis von cli.packPacklist(queryPath, false) sowie die Prüfung von lockFilePath. Überprüfe das Verhalten mit einem Pack, das codeql-pack.lock.yml enthält, und stelle anschließend sicher, dass eine vorhandene Lock-Datei erkannt wird, anstatt immer eine temporäre zu erstellen.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug VSCode

Describe the bug
In src/local-queries/standard-queries.ts , createLockFileForStandardQuery tries to detect whether a pack already has a codeql-pack.lock.yml by checking the output of cli.packPacklist():

const packContents = await cli.packPacklist(queryPath, false);
const lockFilePath = packContents.find((p) =>
    QLPACK_LOCK_FILENAMES.includes(basename(p)),
  );

However, codeql pack packlist doesn't return codeql-pack.lock.yml. This means lockFilePath is always undefined, so the check for an existing lock file can never succeed.

Vorherrschende Sprache
TypeScript
Sterne
539
Forks
240
Ø Merge
2 T. 6 Std.
Gemergte PRs (30 T.)
29

Beitragsleitfaden

Beitragsleitfaden öffnen

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/vscode-codeql

Alle Issues in github/vscode-codeql

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

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