Skip to content

Print debug categories, add examples for fish --debug#7007

Merged
krobelus merged 3 commits into
fish-shell:masterfrom
charego:fish-debugging
May 19, 2020
Merged

Print debug categories, add examples for fish --debug#7007
krobelus merged 3 commits into
fish-shell:masterfrom
charego:fish-debugging

Conversation

@charego
Copy link
Copy Markdown
Contributor

@charego charego commented May 15, 2020

Description

Personally I was most confused by the description of fish --debug-categories and needed to look in the code to figure out it supported comma-separated values in addition to globs. Hopefully this rewording and some examples makes it easier for the next person.

I also found it helpful to print out the enabled debug categories. Without this information it's hard to tell immediately that your --debug-categories value is working as expected.

Copy link
Copy Markdown
Contributor

@krobelus krobelus left a comment

Choose a reason for hiding this comment

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

This is a really useful, especially to have examples!

Comment thread doc_src/cmds/fish.rst Outdated
Comment thread doc_src/cmds/fish.rst Outdated
Comment thread doc_src/cmds/fish.rst Outdated
@krobelus
Copy link
Copy Markdown
Contributor

Seems good.

I guess that most users should never need to use the debug option, but this is probably the most obvious place to add examples to.

@charego
Copy link
Copy Markdown
Contributor Author

charego commented May 17, 2020

Since fish_indent does not support categories, I reverted its documentation.

https://fishshell.com/docs/3.0/commands.html#fish_indent

@krobelus krobelus merged commit b673f32 into fish-shell:master May 19, 2020
@krobelus
Copy link
Copy Markdown
Contributor

Thanks!

@krobelus krobelus added this to the fish 3.2.0 milestone May 19, 2020
@charego charego deleted the fish-debugging branch May 19, 2020 20:13
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants