Hatchlinger Update #475
Hatchlinger Update #475
Conversation
There was a problem hiding this comment.
Dass python 3.8 fälschlicherweise als supported markiert ist, das war schon vor diesem PR so, aber wegen mir könnten wir es hier in diesem PR fixen.
ansonsten sieht's ganz gut aus, aber wir haben v.a. im bereich der docs noch ein zwei "Suchen+Ersetzen" Fehler, wo man nicht einfach setup.py durch pyproject.toml ersetzen kann. Ws. müsstest du wirklich einmal mit annikas hilfe nach RTD reinschauen. Könntest dir vllt ein Pre-release auf diesem branch anlegen und schauen, wie sich das verhält?
docs/conf.py
Outdated
| # DON'T FORGET: Check the box "Install your project inside a virtualenv using | ||
| # setup.py install" in the RTD Advanced Settings. | ||
| # pyproject.toml install" in the RTD Advanced Settings. |
There was a problem hiding this comment.
sicher, dass es die option in RTD gibt?
docs/conf.py
Outdated
| version = "" # Is set by calling `pyproject.toml docs` | ||
| # The full version, including alpha/beta/rc tags. | ||
| release = "" # Is set by calling `setup.py docs` | ||
| release = "" # Is set by calling `pyproject.toml docs` |
There was a problem hiding this comment.
ich glaube nicht, dass das klappt
Co-authored-by: konstantin <konstantin.klein@hochfrequenz.de>
Co-authored-by: konstantin <konstantin.klein@hochfrequenz.de>
There was a problem hiding this comment.
Ich weiß nicht, ob die Docs-Probleme jetzt weg sind oder ob wir die mit hatchlich anders lösen müssen als bisher mit setuptools. Es wäre schön und ich hoffe auch, dass das Löschen der alten Kommentare allein die alten Probleme löst. Andererseits weiß ich, dass da auch viel Schweiß reingeflossen ist, weil es unsere erste RTD-Seite war. Deswegen könnte vllt @hf-aschloegl das noch reviewen?
| # DON'T FORGET: Check the box "Install your project inside a virtualenv using | ||
| # setup.py install" in the RTD Advanced Settings. |
There was a problem hiding this comment.
irgendwie bin ich nicht überzeugt, dass wir das problem lösen, indem wir den "DON'T FORGET" hinweis, der auf das alte setuptools setup verweist einfach löschen. Oder hattest du mit @hf-aschloegl gesprochen oder das explizit getestet?
Falls du es getestet hast, wäre es gut, wenn du an die Stelle schreibst, was das neue Vorgehen ist oder zumindest explizit erwähnst, dass das Problem seit der hatchling umstellung nicht mehr besteht (sofern es eben getestet wurde).
|
Ich gebe Konstantin Recht, man sollte nicht nur die alte Anweisung raus löschen, sondern damit ersetzen, was man stattdessen machen muss. Kevin schaut sich das gerade an und wenn er sein okay gibt, passt das auch für mich. |
|
Ich approve und merge mal den PR damit er durch ist und fixe read the docs in einem extra PR. |
No description provided.