Chewing_Bever
/
backup-tool
1
0
Fork 0
Code Issues 12 Pull Requests 9 Projects 1 Releases Wiki Activity

Ensure we only remove old backups after successfully creating a new one #13

New Issue
Open
opened 2021-04-27 11:01:42 +02:00 by Jef Roosens · 0 comments
Jef Roosens commented 2021-04-27 11:01:42 +02:00
Owner

The specs currently remove the older backups before creating the new one. This creates the problem that a backup can fail, therefore storing one less backup.

This can be fixed quite easily, by just move the remove_backups() call to after the creation of the new one.

The specs currently remove the older backups before creating the new one. This creates the problem that a backup can fail, therefore storing one less backup. This can be fixed quite easily, by just move the `remove_backups()` call to after the creation of the new one.
Jef Roosens added this to the 1.0.0 milestone 2021-04-27 11:01:42 +02:00
Jef Roosens added the
bug
 label 2021-04-27 11:01:42 +02:00
Jef Roosens self-assigned this 2021-04-27 11:01:42 +02:00
Jef Roosens added this to the 1.0.0 project 2021-04-27 11:01:42 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
renovate/flake8-bugbear-22.x
renovate/flake8-comprehensions-3.x
renovate/tox-3.x
renovate/pytest-cov-3.x
renovate/flake8-4.x
renovate/flake8-quotes-3.x
renovate/flake8-bugbear-21.x
renovate/pytest-6.x
renovate/jedi-0.x
develop
recovery-function
master
Labels
Clear labels   
Brainstorm

Place to brainstorm problems or possible ideas   
Parent

Parent issue   
bug

Something is not working   
duplicate

This issue or pull request already exists   
enhancement

New feature   
help wanted

Need some help   
invalid

Something is wrong   
question

More information is needed   
wontfix

This won't be fixed
No Label
Brainstorm
Parent
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No Milestone
1.0.0
Projects
Clear projects
No project
1.0.0
Assignees
Clear assignees
No Assignees
Jef Roosens
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Chewing_Bever/backup-tool#13
There is no content yet.
Delete Branch "%!s(<nil>)"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?