Skip to content

Additional Clean flags to target your cleanup#31

Merged
luthermonson merged 2 commits intogoodhosts:masterfrom
luthermonson:clean-flags
Dec 2, 2021
Merged

Additional Clean flags to target your cleanup#31
luthermonson merged 2 commits intogoodhosts:masterfrom
luthermonson:clean-flags

Conversation

@luthermonson
Copy link
Copy Markdown
Contributor

cleaning everything was a bit too much of a hammer for some people so we've switched to having a bool flag for each clean and an --all or -A to do the original clean with no args we had previously. due to behavior changing from no args doing the full clean to -A doing the full clean we will ++ the minor release when this is merged

vendor includes v0.0.10 bump of hostsfile which includes the windows fixes for 9 hosts/line.

~\goodhosts\cli [clean-flags] > gc hosts
127.0.0.2 b1 a2
127.0.0.1 host host
~\goodhosts\cli [clean-flags] > goodhosts -f ./hosts clean --dry-run --si
127.0.0.1 host host
127.0.0.2 b1 a2
~\goodhosts\cli [clean-flags] > goodhosts -f ./hosts clean --dry-run --sh
127.0.0.2 a2 b1
127.0.0.1 host host
~\goodhosts\cli [clean-flags] > goodhosts -f ./hosts clean --dry-run --rdh
127.0.0.2 b1 a2
127.0.0.1 host
~\goodhosts\cli [clean-flags] > goodhosts -f ./hosts clean --dry-run --rdi
127.0.0.2 b1 a2
127.0.0.1 host host
~\goodhosts\cli [clean-flags] > goodhosts -f ./hosts clean --dry-run --all
127.0.0.1 host
127.0.0.2 a2 b1

@luthermonson luthermonson requested review from Mte90 and tomjn December 1, 2021 17:58
Copy link
Copy Markdown
Contributor

@tomjn tomjn left a comment

Choose a reason for hiding this comment

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

👍🏻

@luthermonson luthermonson merged commit d12ae45 into goodhosts:master Dec 2, 2021
@luthermonson luthermonson deleted the clean-flags branch December 2, 2021 01:39
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