Skip to content
This repository has been archived by the owner on Aug 4, 2023. It is now read-only.

Purge temp files #2

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Purge temp files #2

wants to merge 2 commits into from

Conversation

mycodecrafting
Copy link
Collaborator

No description provided.

chuck-sanders
chuck-sanders reviewed Feb 28, 2022
View reviewed changes
Copy link

@chuck-sanders chuck-sanders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we add the fs.unlink to the catch blocks as well, to make sure we're not leaving junk on errors. Or maybe move it to a finally block.

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

@chuck-sanders chuck-sanders chuck-sanders left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mycodecrafting @chuck-sanders