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

Add "ignoreDirs" option #221

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Add "ignoreDirs" option #221

wants to merge 6 commits into from

Conversation

cgbuen
Copy link
Contributor

@cgbuen cgbuen commented Mar 14, 2017

(Contingent on PR #220 as written.)

Add a CLI/API option to ignore specified directories from being parsed.
This is if the default ignored directories are too aggressive (#208) or
not aggressive enough (#133).

(Perhaps this may also help in avoiding the parse of large directories,
which could contribute to high memory usage described in #175?)

Implement incompete devOnly feature that was documented in the README.
- Remove messaging about / formatting from README autogeneration
- Update API section order consistency with CLI section order
- Update CLI / add API documentation language for "debug" option
- Add missing API documentation for "ignore" option
Add a CLI/API option to ignore specified directories from being parsed.
This if the defaults ignored directories are too aggressive (dylang#208) or
not aggressive enough (dylang#133).

(Perhaps this may also help in avoiding the parse of large directories,
which could contribute to high memory usage described in dylang#175?)
@phdunham
Copy link

phdunham commented May 5, 2017

This is a handy fix. Any chance this will be merged soon?

@ierceg
Copy link

ierceg commented May 17, 2017

Would this be considered for merging soon? We are trying to avoid forking but we really need this or something like it. Thanks in advance.

@fengyuanzemin
Copy link

why this pr not exists in master branch?

I want this option

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

Successfully merging this pull request may close these issues.

4 participants