New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: Allow for shutil.copytree to have existing dirs #156

Merged

matthewfeickert merged 1 commit into main from fix/allow-for-copytree-dirs-to-exist

Jun 17, 2024

Member

matthewfeickert commented Jun 17, 2024 •

edited

Loading

Add dirs_exist_ok=True to allow the copying operation to continue if it encounters existing directories.
- c.f. https://docs.python.org/3.12/library/shutil.html#shutil.copytree
- dirs_exist_ok was added in Python 3.8.
Amends PR fix: Use shutil.copytree over distutils #155.

* Add dirs_exist_ok=True to allow the copying operation to continue if
  it encounters existing directories.
   - c.f. https://docs.python.org/3.12/library/shutil.html#shutil.copytree
   - dirs_exist_ok was added in Python 3.8.
* Amends PR https://github.com/recast-hep/recast-atlas/pull/155.


          fix: Allow for shutil.copytree to have existing dirs

1cb2f61

* Add dirs_exist_ok=True to allow the copying operation to continue if
  it encounters existing directories.
   - c.f. https://docs.python.org/3.12/library/shutil.html#shutil.copytree
   - dirs_exist_ok was added in Python 3.8.
* Amends PR 155.

matthewfeickert added the fix label

matthewfeickert self-assigned this

matthewfeickert merged commit 903b435 into main

10 checks passed

matthewfeickert deleted the fix/allow-for-copytree-dirs-to-exist branch

June 17, 2024 08:34

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix