Skip to content
This repository has been archived by the owner on Sep 19, 2020. It is now read-only.

Clarify exclude path instructions in the CLI help #489

Merged
merged 1 commit into from
Oct 3, 2016

Conversation

unixorn
Copy link
Contributor

@unixorn unixorn commented Oct 3, 2016

When I was beginning to use foodcritic, it was detecting the test subdirectory of some of our cookbooks as another cookbook and throwing false FC011, FC031 and FC045 errors.

When I was trying to exclude the test directory tree, I read the instructions as needing to specify an absolute path, not one relative to the cookbook directory.

This PR spells that out in the --help output.

Closes #488

@tas50 tas50 merged commit 565c0c0 into Foodcritic:master Oct 3, 2016
@tas50
Copy link
Contributor

tas50 commented Oct 3, 2016

Certainly worth adding there. Thanks

@unixorn
Copy link
Contributor Author

unixorn commented Oct 3, 2016

You're welcome. That was a smoking fast merge :-)

@unixorn unixorn deleted the clarify-exclude-instructions branch October 3, 2016 18:35
@tas50 tas50 changed the title Clarify exclude path instructions Clarify exclude path instructions in the CLI help Oct 20, 2016
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.

None yet

2 participants