Skip to content
This repository has been archived by the owner on Jun 9, 2021. It is now read-only.

update usage information #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rogpeppe
Copy link
Collaborator

This brings README.md and the usage printed by the command
into sync, and also adds information about available attributes.

I've reverse-engineered the descriptions from my idea of what the
code might be doing, so they're quite likely wrong :)

This brings README.md and the usage printed by the command
into sync, and also adds information about available attributes.

I've reverse-engineered the descriptions from my idea of what the
code might be doing, so they're quite likely wrong :)
Copy link
Owner

@mvdan mvdan left a comment

Choose a reason for hiding this comment

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

Thanks! Just one nit and one bit of context.

The node is an identifier that matches the regular expression.
type(type)
The node has the given Go type.
TODO how are packages referred to?
Copy link
Owner

Choose a reason for hiding this comment

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

packages are referred to as per the scope where the node is located, if I recall correctly.

I also seem to recall this is an area that I wasn't too happy with. The ideal solution might be local scope plus some sort of fallback, like goimports-like resolution. Or perhaps something more explicit, like what rsc/rf does.

@@ -17,6 +17,7 @@ A command is of the form "-A pattern", where -A is one of:
-v discard nodes matching a pattern
-a filter nodes by certain attributes
-s substitute with a given syntax tree
-p number navigate up a number of node parents
Copy link
Owner

Choose a reason for hiding this comment

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

Keep the help texts aligned here?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants