Repository navigation
docs(marketing): bilingual sequence diagrams, test & style badges, and SEO keywords [G 2026-11-08] - #6
Open
lukisch wants to merge 1 commit into
Conversation
…d SEO keywords [G 2026-11-08]
|
Thank you for your pull request! The team will inspect and review your contribution. |
lukisch
commented
Nov 11, 2026
lukisch
left a comment
Contributor
Author
There was a problem hiding this comment.
Zweitprüfung auf 64a77ef: keine Blocker, eine Zahl sollte vor dem Merge korrigiert werden. (Als Kommentar, weil GitHub eine formale Freigabe des eigenen PRs nicht zulässt.)
Alle 7 Checks am Head sind grün. Lokal konnte ich die Suite nicht ausführen (dem Container fehlt libEGL für PySide6); die Bewertung stützt sich auf den Diff, den Quelltext und das CI-Log.
Bitte korrigieren
- Das neue Badge nennt 180 bestandene Tests, der CI-Lauf dieses PRs meldet 179 passed, 1 skipped (Job
pytest (3.11)). Der neue Vertragstest schreibt die 180 zusätzlich fest. Vorschlag: Badge und Test auf179 passed | 1 skippedstellen, oder die Zahl ganz aus dem Badge nehmen und auf das CI-Badge verweisen. Der Changelog-Eintrag („auf 180 bestandene Tests aktualisiert“) wäre dann mit anzupassen.
Geprüft und in Ordnung
- Die vier neuen Badge-Anker (
#tests--quality-verification,#local-data-flow--privacy-isolationund die beiden deutschen) existieren in beiden READMEs. - Das Diagramm passt zum Code in
PDFtoPDFocr_2.py:convert_from_pathfür PDFs,Image.alpha_compositeauf weißem Hintergrund für Bilder, Ausgabe als<stem>_ocred.pdf, ShortcutCtrl+Return, Zusammenbau mit pikepdf. - Die Schlussnotiz nennt den möglichen GitHub-Abruf fehlender Sprachdaten. Das Badge „local-first“ behauptet damit nicht mehr als der Fließtext.
Hinweise, nicht blockierend
- Im Diagramm steht „Return OCR text and hOCR / PDF layers“. Der Code ruft
image_to_pdf_or_hocr(..., extension='pdf')auf, es entstehen also nur PDF-Seiten, kein hOCR. Entsprechend fügt pikepdf fertige OCR-Seiten zusammen und „injiziert“ keine Textebene in ein bestehendes PDF. CHANGELOG.mdenthält den Abschnitt## [Unreleased]zweimal (Zeile 6 und ab Zeile 161). Das bestand schon vorher; dieser PR trägt den neuen Eintrag in beide Kopien ein. Die Dublette sollte bei Gelegenheit entfernt werden.
Generated by Claude Code
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.
Summary
Turnusgemäßer Pfad-B-Discoverability-, zweisprachiger README-Architektur-, Sequenzdiagramm- und SEO-Erweiterungslauf für doc-bricks/PDFtoPDFocr auf WORKSTATION-LG durch Gemini (Antigravity worker, Flash High).
Changes
uff check .\ (all checks passed).