Seite 1 von 1
nvtransfer - trägt eine virtuelle Umgebung von einem Arbeits
Verfasst: Montag 8. Juni 2015, 08:03
von rand007
Envtransfer - trägt eine virtuelle Umgebung von einem Arbeitsplatz zum anderen. Vorschau auf die Anwendung und deren Quellcode, können Sie auf der Projektseite.
https://github.com/deslum/envtransfer
Warten auf Feedback, Kritik und Anregungen. Danke.
Re: nvtransfer - trägt eine virtuelle Umgebung von einem Arb
Verfasst: Montag 8. Juni 2015, 08:26
von Sirius3
@rand007: die git-Commit-Messages sind sehr nichtssagend. Da könnte man sie gleich weglassen, da das Datum eh mitgeloggt wird.
Nur mal kurz in core.py reingeschaut:
- Zeile 23: join ist eigentlich für Listen da, nicht für pseudo-Listen, hier ist Stringformatierung besser.
- Zeile 26: bei POST-requests wird automatisch Content-Type und Content-Length gesetzt. Host wird sowieso gesetzt, das header-Argument ist also unnötig.
- Zeile 33, 46, 56, 71, ...: wieder falsch eingesetztes join
- Zeile 37ff: find_file ist ein irreführender Funktionsname. Funktionsweise völlig undurchsichtig. Globale Variablen vermeiden!
Insgesamt könnte sich ein Blick auf das requests-Modul lohnen.
Die anderen Module sind alle sehr klein und es ist eigentlich nicht nötig, dafür eigene Module zu bauen.
Re: nvtransfer - trägt eine virtuelle Umgebung von einem Arb
Verfasst: Montag 8. Juni 2015, 08:34
von jens
Eigentlich witzige Idee.... Aber was ist "Yandex disk" ? Vielleicht in der README erklären/verlinken?
btw. in der README auch erwähnen, in welchen Fällen es überhaupt funktioniert. Von wegen selbe Platform, selbe Python Version usw.
Und
https://travis-ci.org/deslum/envtransfer/ ist momentan noch ziemlich Sinnlos. Wird doch nur festgestellt, das es generell installiert werden kann.