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

Hide macro from docs #2430

Closed
alf-mindshift opened this issue May 12, 2020 · 1 comment · Fixed by #2431
Closed

Hide macro from docs #2430

alf-mindshift opened this issue May 12, 2020 · 1 comment · Fixed by #2431
Labels
dbt-docs [dbt feature] documentation site, powered by metadata artifacts enhancement New feature or request good_first_issue Straightforward + self-contained changes, good for new contributors!

Comments

@alf-mindshift
Copy link
Contributor

Describe the feature

We have some macros in our project that are not relevant to the consumers of our documentation site. Issue #1671 and pr #2179 came very close to delivering the necessary functionality to hide them.

Describe alternatives you've considered

We can post-process the manifest.json file instead. This would be annoying and only fix the problem for us. A general solution would be better.

Additional context

This is a trivial change.

Who will this benefit?

This makes it easier to tweak the documentation site to work for less technical people.

@drewbanin
Copy link
Contributor

Thanks for opening this issue @alf-mindshift. I think the root issue here is that the show config is not compiled into the manifest.json file. Once we make that change, we can make the corresponding change in the docs website to remove these hidden nodes from the file tree.

@drewbanin drewbanin added dbt-docs [dbt feature] documentation site, powered by metadata artifacts good_first_issue Straightforward + self-contained changes, good for new contributors! labels May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dbt-docs [dbt feature] documentation site, powered by metadata artifacts enhancement New feature or request good_first_issue Straightforward + self-contained changes, good for new contributors!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants