Skip to content

Hide show mount point - #107

Merged
muesli merged 23 commits into
muesli:masterfrom
IGLOU-EU:adrien/HideShow_mountPoint
Apr 21, 2021
Merged

muesli merged 23 commits into
muesli:masterfrom
IGLOU-EU:adrien/HideShow_mountPoint

Conversation

@IGLOU-EU

Copy link
Copy Markdown
Contributor

Fix #101

@luckman212

Copy link
Copy Markdown

Possible to add wildcard support?

duf -hide-mp '/Volumes/.timemachine*'

or

duf -only-mp '/System/Volumes/*'

@IGLOU-EU

Copy link
Copy Markdown
Contributor Author

Hi @luckman212
this is a good idea, I can do that.

@IGLOU-EU IGLOU-EU changed the title Adrien/hide show mount point WIP hide show mount point Feb 24, 2021
@IGLOU-EU IGLOU-EU changed the title WIP hide show mount point Hide show mount point Mar 27, 2021
@IGLOU-EU

Copy link
Copy Markdown
Contributor Author

Add * support for pattern matching
Requested by #106

@muesli muesli left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Nice! Sorry for all the nitpicky typo comments 😃

Comment thread main.go Outdated
Comment thread main.go Outdated
Comment thread main.go Outdated
Comment thread main.go Outdated
Comment thread main.go Outdated
Comment thread main.go Outdated
Comment thread main.go Outdated
@muesli

muesli commented Mar 28, 2021

Copy link
Copy Markdown
Owner

I'm going to rebase master on top of your work, in the hope that it fixes the CI error.

@IGLOU-EU

Copy link
Copy Markdown
Contributor Author

Nice! Sorry for all the nitpicky typo comments 😃

Ho ... I'm so sorry about that 😖

@IGLOU-EU
IGLOU-EU requested a review from muesli March 28, 2021 12:56
Comment thread go.sum Outdated
@IGLOU-EU

IGLOU-EU commented Apr 2, 2021

Copy link
Copy Markdown
Contributor Author

this is ok for you @muesli ?

@IGLOU-EU
IGLOU-EU requested a review from muesli April 2, 2021 22:43
@muesli

muesli commented Apr 3, 2021

Copy link
Copy Markdown
Owner

Will give it one final review today!

@muesli

muesli commented Apr 9, 2021

Copy link
Copy Markdown
Owner

Ok, played with this and it's looking good. However on my first try I fell for a trap:

duf --only-mp /foobar/*

This of course uses the shell's globbing and delivers unexpected results. I'm thinking we'll need to provide some examples in the README and point out that caveat, to avoid confusion. I'll take care of this and push some updates to your branch, which should finally finish this up.

Thank you for your contribution and all the work involved!

Comment thread go.mod Outdated
@IGLOU-EU
IGLOU-EU requested a review from muesli April 9, 2021 11:47
@IGLOU-EU

Copy link
Copy Markdown
Contributor Author

Thank you for your time and review. Many would like to get more involved in Free Software, and it's with people like you that we can :)

@muesli

muesli commented Apr 14, 2021

Copy link
Copy Markdown
Owner

Thank you for your time and review. Many would like to get more involved in Free Software, and it's with people like you that we can :)

I'm glad to hear you enjoy contributing to open-source! Looking forward to many more contributions!

Will give this a final review before this weekend and merge it!

@muesli muesli left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM! Thank you!

IGLOU-EU added 10 commits April 21, 2021 10:37
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
IGLOU-EU and others added 13 commits April 21, 2021 10:44
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
Signed-off-by: Adrien Kara <adrien@iglou.eu>
@muesli muesli added the enhancement New feature or request label Apr 21, 2021
@muesli
muesli merged commit a200812 into muesli:master Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Option to hide a named mount point?

3 participants