Suggestion: have repr(oid) be `Oid(hex='a1b2c3')` instead of `a1b2c3`
Offen
Anfängerfreundlich
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Python
- Sterne
- 1.7k
- Forks
- 408
- Ø Merge
- 2 T. 57 Min.
- Gemergte PRs (30 T.)
- 7
Beschreibung
I suggest to make the repr of Oid objects be:
Oid(hex='a1b2c3')
instead of
a1b2c3
I think that it's better if repr returns a valid python expression. It's clearer what the object is, and it makes it easier to reconstruct it.
I can probably submit a pull request if this is approved.
Thanks!
Noam
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die Python-Oid-Implementierung und ihr repr-Verhalten zu lokalisieren, und sieh dir anschließend die Tests in der Nähe für Oid-Stringdarstellungen an. Aktualisiere die Darstellung so, dass sie ein gültiger Ausdruck Oid(hex='...') ist, und bestätige, dass die vorhandenen Darstellungstests die neue Ausgabe abdecken.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- python
- Bereich
- api
- Issue-Typ
- Feature
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Aktiv
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 72/100