Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The target list option should include hidden targets #10

Open
pkar opened this issue Feb 24, 2017 · 2 comments
Open

The target list option should include hidden targets #10

pkar opened this issue Feb 24, 2017 · 2 comments

Comments

@pkar
Copy link
Collaborator

pkar commented Feb 24, 2017

Right now it does not display them because underscore targets are hidden.

$ ron t _go
_go_bench          _go_build_all      _go_lint           _go_tag            _go_testv          _go_vendor_update
_go_build          _go_install        _go_race           _go_test           _go_vendor         _go_vet
$ ron t -l _go*
Targets:


Default Targets:
@noonat
Copy link
Member

noonat commented Feb 27, 2017

I think that was intentional, that underscore prefixed targets were intended to be "internal" targets called by other targets, not intended for running directly. I think that usage ended up getting combined with prefixing default targets, because everything is in the global namespace... I imagine these will be go:bench go:build etc. once #11 is done.

Maybe we should preserve the behavior of hiding underscored names by default so that they can continue to be used for internal targets, and just add a command line flag to show all targets?

@pkar
Copy link
Collaborator Author

pkar commented Mar 6, 2017

It was intentional. I think it's simpler than having another flag. If the search term begins with an _ then start displaying those

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

No branches or pull requests

2 participants