Repository navigation
Logo über next/image, Lizenzprüfung als Positivliste, Actions aktualisiert - #34
Merged
Merged
Conversation
The header and footer showed the full 678 px, 176 KB logo.png at 32 to 36 px. next/image sends a scaled WebP instead, which also clears the no-img-element warning from the code scan.
Dependency ReviewThe following issues were found:
License Issues.github/workflows/eslint.yml
.github/workflows/secret-scanning.yml
OpenSSF Scorecard
Scanned Files
|
The dependency review action deprecated deny-licenses (actions/dependency-review-action#997). A deny list of GPL-2.0 and GPL-3.0 also let AGPL and every other license through. The allow list holds what the lockfile has today plus AGPL-3.0-or-later, the project's own license. checkout and setup-node move to v7; neither change touches these workflows (no pull_request_target, no registry-url).
The delete trigger started a run for every branch a merged pull request or Dependabot removed. The next push to main prunes such a branch on Codeberg anyway.
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.
Header und Footer haben das volle logo.png geladen (678 × 671 px, 176 KB), obwohl es nur mit 32 bis 36 px angezeigt wird. Mit next/image kommt eine passend verkleinerte WebP-Datei, dazu ein srcset für 2x. Damit sind auch die beiden Warnungen @next/next/no-img-element aus dem Code-Scan von #31 erledigt. Im Header lädt das Logo mit loading="eager", weil es immer sichtbar ist.
Die Dependency Review hat gewarnt, dass deny-licenses wegfällt (actions/dependency-review-action#997). Statt der Sperrliste GPL-2.0, GPL-3.0 steht jetzt eine Positivliste drin: alles, was heute im Lockfile vorkommt, plus AGPL-3.0-or-later als eigene Lizenz des Projekts. Die alte Liste hätte zum Beispiel AGPL durchgelassen.
actions/checkout steht überall auf v7 (codeql.yml hatte noch v4), actions/setup-node auf v7. Die Änderungen dort betreffen uns nicht, wir nutzen weder pull_request_target noch registry-url.
Geprüft: tsc und eslint lokal sauber, alle Workflows parsen als YAML.
Mirror to Codeberg läuft nur noch bei Pushes auf main und bei Tags. Der Trigger delete hat für jeden Branch, den ein gemergter PR oder Dependabot gelöscht hat, einen eigenen Lauf gestartet. Aufgeräumt wird auf Codeberg trotzdem, der nächste Push auf main entfernt den Branch dort mit --prune.