Update and enhance documentation
#100
Merged
christian.merten
merged 4 commits from ku/documentation into main 11 months ago
Reviewers
Request review
No reviewers
Labels
Clear labels
Betrifft die Administration des Kompass
Betrifft Anwender:innen des Kompass, das heißt nicht-Admins
Betrifft die Entwicklungs- und Produktiv Infrastruktur
Bedarf keiner strukturellen Änderungen und ist leicht umsetzbar, weil es ähnliche Funktionen schon gibt
Breaking change that won't be backward compatible
Something is not working
Documentation changes
Improve existing functionality
New functionality
This is security issue
Issue or pull request related to testing
Betrifft die Kompass Produktivumgebung von Heidelberg
Betrifft die Kompass Produktivumgebung von Ludwigsburg
The priority is critical
The priority is high
The priority is low
The priority is medium
Issue has been confirmed
This issue or pull request already exists
Invalid issue
This issue won't be fixed
Somebody has started to work on this but abandoned work
The pull request or issue awaits a response by the author to questions or comments by reviewers.
Something is blocking this issue or pull request
Feedback is required to reproduce issue or to continue work
Apply labels
Effekt
Admins
Betrifft die Administration des Kompass
Effekt
Anwender
Betrifft Anwender:innen des Kompass, das heißt nicht-Admins
Effekt
Infrastruktur
Betrifft die Entwicklungs- und Produktiv Infrastruktur
Issue gut geeignet für Einsteiger:innen
Bedarf keiner strukturellen Änderungen und ist leicht umsetzbar, weil es ähnliche Funktionen schon gibt
Kind/Breaking
Breaking change that won't be backward compatible
Kind/Bug
Something is not working
Kind/Documentation
Documentation changes
Kind/Enhancement
Improve existing functionality
Kind/Feature
New functionality
Kind/Security
This is security issue
Kind/Testing
Issue or pull request related to testing
Lokal
Heidelberg
Betrifft die Kompass Produktivumgebung von Heidelberg
Lokal
Ludwigsburg
Betrifft die Kompass Produktivumgebung von Ludwigsburg
Priority
Critical
The priority is critical
Priority
High
The priority is high
Priority
Low
The priority is low
Priority
Medium
The priority is medium
Reviewed
Confirmed
Issue has been confirmed
Reviewed
Duplicate
This issue or pull request already exists
Reviewed
Invalid
Invalid issue
Reviewed
Won't Fix
This issue won't be fixed
Status
Abandoned
Somebody has started to work on this but abandoned work
Status
Awaiting author
The pull request or issue awaits a response by the author to questions or comments by reviewers.
Status
Blocked
Something is blocking this issue or pull request
Status
Need More Info
Feedback is required to reproduce issue or to continue work
No Label
Effekt
Admins
Effekt
Anwender
Effekt
Infrastruktur
Issue gut geeignet für Einsteiger:innen
Kind/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Lokal
Heidelberg
Lokal
Ludwigsburg
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Awaiting author
Status
Blocked
Status
Need More Info
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Set Project
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
No dependencies set.
Reference: digitales/kompass#100
Reference in New Issue
There is no content yet.
Delete Branch 'ku/documentation'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
Ich habe einige Links aus Deiner @christian.merten ursprünglichen Nutzer Doku umbenannt, die sollten alle wieder funktionieren.
Update and enhance documentationto WIP: Update and enhance documentation 11 months agoWIP: Update and enhance documentationto Update and enhance documentation 11 months agoIch bekomme
Hast du die geupdatede
requirements.txtin einem cleanen docker build getestet?Das hatte ich tatsächlich nicht getestet. Bei mir hat es im docker auch nicht geklappt. Das liegt an der python Version. Lokal habe ich 3.11 und der docker container nutzt 3.9.
Ich habe jetzt mal lokal das docker image auf
python:3.11-bullseyegesetzt, dann klappt der build problemlos.Ich bin mir aber nicht sicher, ob ich das allgemein machen kann oder das dann wo anders Probleme verursacht.
Hmm, mit python version auf 3.11 hochsetzen wäre ich vorsichtig, da müsste man schon erstmal ausgiebig testen (und leider ist die test coverage zur Zeit so niedrig, dass ich da nicht viel Sicherheit ziehen würde). Ist die höhere sphinx Version zwingend notwendig?
Ich hatte die hochgesetzt, weil ich zwischendurch über ein Feature gestolpert bin, dass es erst ab 8.1 gab.
Jetzt weiß ich aber leider nicht mehr welches das war... Ich guck nochmal.
Lokal scheint es mir auch mit sphinx 7.4.7 ohne Einschränkungen zu klappen.
Im docker kann ich es gerade leider nicht testen, weil das development image die doku nicht served (oder?) und ich für das production image keine funktionierende .env Datei habe.
Aber es wird auf jeden Fall mit dieser Version in beiden images erfolgreich gebaut und der html code generiert.
Top, kannst du den PR updaten? Dann teste ich es.
Du kannst die html Dateien, die im development image generiert werden, lokal im Browser öffnen (also direct die Dateien unter
docs/build/).Also die vom development build generierte Doku in
docs/build/sieht vollständig und korrekt aus.Sphinx version habe ich wieder gedowngraded.
Vielen Dank!
Was ich noch nicht ganz verstehe: Wieso der
_staticfolder? Das sieht so automatisch generiert aus. Vielleicht einfachstatic?@ -10,3 +14,2 @@copyright = '2024, Christian Merten'author = 'Christian Merten'release = '2.0'author = 'Christian Merten'Da würde ich einfach schreiben
'The Kompass team'oder so.@ -11,2 +14,3 @@author = 'Christian Merten'release = '2.0'author = 'Christian Merten'copyright = f'%Y, {author}'Das
'%Y'scheint nicht zu funktionieren, wird einfach als%Yangezeigt. Würde ich daher einfach manuell auf2025setzen.Ah, ich erinnere mich wieder. Das war das Feature, das erst ab sphinx 8.1 geht. Aber das ist nicht so wichtig, das geht zunächst auch hardcoded.
@ -0,0 +36,4 @@Workflow--------- request a gitea account with ChristianDa würde ich einfach
from the maintainers.schreiben._staticist der default von sphinx selber: https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-html_static_pathDas ist aber eigentlich egal, wir können das selber festlegen.
Es setzt sich wahrscheinlich besser ab, als die restlichen Ordner, die nur Inhalt haben und ohne führenden
_geschrieben werden. Der Templates Ordner wird teilweise auch mit_templatesgeschrieben. Wobei im der Doku wird er mit.templatesvorgeschlagen.Danke!
35e565d0bdinto main 11 months agoReviewers
35e565d0bd.