Replace some os.rename with shutil.move (!818) · Merge requests · lscsoft / bilby · GitLab

Snippets Groups Projects

Maintenance will be performed on git.ligo.org, containers.ligo.org, and docs.ligo.org on Tuesday 22 April 2025 starting at approximately 9am PDT. It is expected to take around 30 minutes and there will be several periods of downtime throughout the maintenance. Please address any comments, concerns, or questions to the helpdesk. This maintenance will be upgrading the GitLab database in order to be ready for the migration.

Merged Moritz Huebner requested to merge make_safe_file_dump_safer into master 4 years ago

Windows does not allow overwriting files with os.rename. shutil.move makes things safe.

Edited 4 years ago by Moritz Huebner

Approval is optional

Activity

Moritz Huebner changed milestone to %0.6.10 4 years ago

changed milestone to %0.6.10
Moritz Huebner added Bug label 4 years ago

added Bug label
Moritz Huebner added 1 commit 4 years ago
added 1 commit

5548ef0a - replaced usage of os.remove/replace with shutil.move

Compare with previous version
Moritz Huebner changed the description 4 years ago

changed the description
Moritz Huebner changed milestone to %Future 4 years ago

changed milestone to %Future
Matthew Pitkin approved this merge request 4 years ago

approved this merge request
Sylvia Biscoveanu approved this merge request 4 years ago

approved this merge request
Moritz Huebner changed milestone to %1.0.1 4 years ago

changed milestone to %1.0.1
Gregory Ashton changed milestone to %1.0.0 4 years ago

changed milestone to %1.0.0
Gregory Ashton merged 4 years ago

merged
Gregory Ashton mentioned in commit 5f37ded2 4 years ago

mentioned in commit 5f37ded2

Please register or sign in to reply