drivendataorg/cloudpathlib

Add a `dirs_exist_ok` parameter to the `copytree` implementation

Offen

#144 geöffnet am 20.05.2021

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Python (87 Forks)auto 404
enhancementgood first issue

Repository-Metriken

Stars
 (624 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

We could add a dirs_exist_ok parameter to copy tree. To do this, we would also need to raise on mkdir if dir exists for cloud implementations. For most clients mkdir is a no-op right now, so that logic would need to be added/tested.

It seems like shutil.copytree has a dirs_exist_ok parameter that controls this exist_ok. Do we want to mimic that interface?

https://docs.python.org/3/library/shutil.html#shutil.copytree

Originally posted by @jayqi in https://github.com/drivendataorg/cloudpathlib/pull/142#discussion_r634894797

Contributor Guide

Research-Richtung
Implementieren Sie einen `dirs exist ok` Parameter in der `copytree` Methode, ändern Sie die `mkdir` Aufrufe so, dass sie eine Ausnahme auslösen, wenn das Verzeichnis existiert und `exist ok` False ist.
Tech Stack
python
Domain
backend
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Python
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65