Repository navigation
Security-Review baut die Pakete, bevor es testet - #30
Merged
Merged
Conversation
Das erste Review mit Tests (opengewerk/opengewerk-haustechnik#223) blieb statisch, weil die Pakete des Arbeitsbereichs nicht gebaut waren und die Tests sie aus ihren Builds importieren. Für ein Repository mit npm statt pnpm installiert der Lauf jetzt auch mit npm ci.
Musiker15
added a commit
to opengewerk/opengewerk-haustechnik
that referenced
this pull request
Oct 10, 2026
* Security-Review: Aufruf auf den Stand mit Bau heben Der Workflow der Organisation baut seit opengewerk/.github#30 die Pakete, bevor er testet. Die ersten Reviews hier blieben ohne sie statisch. * Security-Review: Aufruf auf den gehärteten Stand heben Der Workflow der Organisation (opengewerk/.github#31) führt keinen Code des PRs mehr außerhalb der Sandbox aus, schreibt nichts und legt vorbestehende Lücken als Code-Scanning-Alerts ab. Der Aufruf gibt dafür security-events mit; die Testdatenbank entfällt.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Das erste volle Review mit Tests (opengewerk/opengewerk-haustechnik#223) blieb rein statisch: der Bericht sagt, dass die Tests nicht liefen, weil @opengewerk/haustechnik-catalogue/testing nicht gebaut war. Die Tests importieren die Pakete des Arbeitsbereichs aus ihren Builds, wie in der CI der Repositories.
Danach heften die fünf Aufrufe diesen Stand an.