Skip to content

Comments

Use rm -rf to remove fixtures#2043

Merged
SuperQ merged 1 commit intoprometheus:masterfrom
luizluca:fixtures_readonly_files
May 20, 2021
Merged

Use rm -rf to remove fixtures#2043
SuperQ merged 1 commit intoprometheus:masterfrom
luizluca:fixtures_readonly_files

Conversation

@luizluca
Copy link
Contributor

rm interactively asks for confirmation while removing read-only
files and fixtures sys has dozens of them.

Signed-off-by: Luiz Angelo Daros de Luca luizluca@gmail.com

rm interactively asks for confirmation while removing read-only
files and fixtures sys has dozens of them.

Signed-off-by: Luiz Angelo Daros de Luca <luizluca@gmail.com>
Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@SuperQ
Copy link
Member

SuperQ commented May 20, 2021

Yea, I've noticed this before.

@SuperQ SuperQ merged commit 220aa5b into prometheus:master May 20, 2021
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.

2 participants