Skip to content

Conversation

slice4e
Copy link
Collaborator

@slice4e slice4e commented Nov 24, 2022

Fixes #87

It seems that we were deleting "temporary_dir" but not "temporary_dir_client"
This should be fixed now ..

@filipecosta90
Copy link
Contributor

@slice4e please format it:

1 file would be reformatted, 38 files would be left unchanged.

you can do it via:

poetry run black . 

Copy link
Contributor

@filipecosta90 filipecosta90 left a comment

Choose a reason for hiding this comment

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

need to reformat using black.

@slice4e
Copy link
Collaborator Author

slice4e commented Nov 25, 2022

@slice4e please format it:

1 file would be reformatted, 38 files would be left unchanged.

you can do it via:

poetry run black . 

Done

@filipecosta90 filipecosta90 merged commit d6be093 into redis:main Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG]: redis-benchmarks-spec-sc-coordinator is not removing temporary folders after un
2 participants