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

Support for excluding modules? #42

Closed
oubiwann opened this issue Jan 18, 2016 · 7 comments
Closed

Support for excluding modules? #42

oubiwann opened this issue Jan 18, 2016 · 7 comments

Comments

@oubiwann
Copy link
Contributor

There are some modules I won't want included in a public API (e.g., modules that bring in functions defined in C ports, for which there are friendlier, "public" modules).

Could you add a configuration file (or flag) option for excluding modules from the generated docs?

@yurrriq
Copy link
Member

yurrriq commented Jan 18, 2016

I'm working on a configuration revamp right now. How would you feel about something like this in the config:

{'excluded-modules',
  ['secret-mod', 'private-mod', 'excluded-mod']}

^ a list of module names to exclude.

@oubiwann
Copy link
Contributor Author

Yeah, that's the sort of thing I was thinking.

If this goes in the rebar.config file (possible? I hope so...), I'd add lodox context, e.g.:

{lodox [{'excluded-modules',
         ['secret-mod','private-mod','exlcuded-mod']}]}.

This leaves a whole bunch of nice possibilities for lodox in the future (configurability for things we can't think of right now).

If rebar3 barfs on having extraneous data in rebar.config, I'd opt for using lfe.config ... since it'd be a shame to have a proliferation of configuration files in LFE projects ...

@yurrriq
Copy link
Member

yurrriq commented Jan 18, 2016

That's exactly what I've got cooking. 😄

@oubiwann
Copy link
Contributor Author

SWEEEEET! :neckbeard: ❤️

@yurrriq yurrriq added this to the 1.0.0 milestone Jan 19, 2016
yurrriq added a commit that referenced this issue Jan 21, 2016
@yurrriq
Copy link
Member

yurrriq commented Feb 28, 2016

This needs to be cleaned up and documented, but the basic functionality is there.

@yurrriq
Copy link
Member

yurrriq commented Feb 29, 2016

@yurrriq
Copy link
Member

yurrriq commented Apr 26, 2016

Moved to lfe-rebar3#18.

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

No branches or pull requests

2 participants