Move `creation_callback` arg to be final arg of `generate` function

Offen
#9 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
45/100
Issue-Typ
Refactoring
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
python
Bereich
tooling

Rechercherichtung

Beginne in python/openassetio_traitgen/main.py bei der Funktion generate etwa in Zeile 179, wo creation_callback definiert ist. Untersuche, wie die Funktion aufgerufen wird, und bestätige dann, dass der Callback das letzte Argument ist und dass nicht angegebenes Logging standardmäßig auf stderr erfolgt; abgeschlossen ist die Aufgabe, wenn die Generierung mit den überarbeiteten Argumenten weiterhin funktioniert.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement

Spawned from this conversation.

Move the creation_callback argument in the generate method to the end of the function, so users can opt to omit it more often.

The reasoning is that it's rarely used, and it's a more complicated type than your bog standard ints and bools, so isn't very intuitive.

Default logging to std::err if non supplied.

Vorherrschende Sprache
Python
Sterne
7
Forks
7
Ø Merge
1 Std. 7 Min.
Gemergte PRs (30 T.)
1

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus OpenAssetIO/OpenAssetIO-TraitGen

Alle Issues in OpenAssetIO/OpenAssetIO-TraitGen

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.