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

Report skipped and included declarations (take 2) #811

Closed
wants to merge 2 commits into from

Conversation

johnfairh
Copy link
Collaborator

This adds a report to stdout about the number of declarations included/skipped in the docs, pretty much per #238, eg.

included 964 public or open symbols
skipped 189 private, fileprivate, or internal symbols (use `--min_acl` to see)

New users are still having acl problems with their first use of jazzy, maybe this will help some.

Added a specs change to match for CI -- will redo when Xcode 83 stuff is in. No rush, already two years since this issue was opened...

@jpsim
Copy link
Collaborator

jpsim commented May 11, 2017

Merged in e440870 & 88dd1a0 with super minor tweaks. Thanks!

@jpsim jpsim closed this May 11, 2017
@jpsim jpsim deleted the issue-238-2 branch May 11, 2017 23:00
@johnfairh
Copy link
Collaborator Author

Cheers - 👍 tweaks.

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.

2 participants