fix: formulation template name #132
Open
Conversation
This file contains 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
larsbonczek
reviewed
Oct 29, 2024
There was a problem hiding this comment.
Ich bin mir nicht sicher, ob es sinnvoll ist, das so zu lösen. Das minimal example soll ja als Orientierung für Paketentwickler*innen dienen. Aber ist das mit diesen automatisch berechneten Konstanten wirklich das, was wir als Standard-Vorgehen im QuestionPy-Ökosystem sehen wollen? Ich denke es wäre besser wenn entweder
- der Namespace und Kurzname beim Erzeugen des Beispiel-Pakets in der entsprechenden Zeile eingefügt werden (das klappt im Manifest ja auch), oder
- die
get_template-Funktion ohne Angabe des Namespace und Kurznamen aufgerufen werden kann und dann per Default im Template-Verzeichnis des eigenen Pakets sucht.
tumidi
reviewed
Oct 31, 2024
| # Rename template name. | ||
| with FileInput(short_name_folder / "question_type.py", inplace=True) as file: | ||
| for line in file: | ||
| if file.filelineno() == 20: # noqa: PLR2004 |
There was a problem hiding this comment.
Die Zeilennr. hardcoden kann's irgendwie nicht sein. Das geht kaputt sobald jemand die Datei question_type.py mal anfassen muss.
|
Meine 2 Cents zu dem Thema: Ich würde ja das project scaffolding komplett von den examples trennen. Dafür gibt's ja eigentlich ganz gute Lösungen: z.B. Cookiecutter. Da schreibt man letztendlich jinja2-Templates.
|
larsbonczek
requested changes
Nov 1, 2024
| # Rename template name. | ||
| with FileInput(short_name_folder / "question_type.py", inplace=True) as file: | ||
| for line in file: | ||
| if file.filelineno() == 20: # noqa: PLR2004 |
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.
Fixes #130.
PS: Ein
git pullreicht nicht aus, damit dercreate-Befehl die Änderungen zeigt. Es muss vorher nochpoetry installausgeführt werden, damit dieminimal_example.zipneu generiert wird.