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

docs: makes all publicly available types viewable in docs #507

Merged
merged 1 commit into from
May 14, 2016
Merged

Conversation

kbknapp
Copy link
Member

@kbknapp kbknapp commented May 13, 2016

Some types weren't viewable in the docs, such as Values, OsValues,
and ArgSettings. All these types should now be browsable in the
docs page.

Relates to #505

Some types weren't viewable in the docs, such as `Values`, `OsValues`,
and `ArgSettings`. All these types should now be browsable in the
docs page.

Relates to #505
@yo-bot
Copy link

yo-bot commented May 13, 2016

r? @sru

(yo-bot has picked a reviewer for you, use r? to override)

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 88.768% when pulling 52ca650 on issue-505 into 0c6c4ad on master.

@kbknapp
Copy link
Member Author

kbknapp commented May 14, 2016

@homu r+

@homu
Copy link
Contributor

homu commented May 14, 2016

📌 Commit 52ca650 has been approved by kbknapp

homu added a commit that referenced this pull request May 14, 2016
docs: makes all publicly available types viewable in docs

Some types weren't viewable in the docs, such as `Values`, `OsValues`,
and `ArgSettings`. All these types should now be browsable in the
docs page.

Relates to #505
homu added a commit that referenced this pull request May 14, 2016
docs: makes all publicly available types viewable in docs

Some types weren't viewable in the docs, such as `Values`, `OsValues`,
and `ArgSettings`. All these types should now be browsable in the
docs page.

Relates to #505
@homu
Copy link
Contributor

homu commented May 14, 2016

⌛ Testing commit 52ca650 with merge d392fd0...

@homu
Copy link
Contributor

homu commented May 14, 2016

☀️ Test successful - status

@homu homu merged commit 52ca650 into master May 14, 2016
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.

5 participants