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

force unwrap to remove Optional in front of two markdown headers #2272

Open
wants to merge 1 commit into
base: tidepool-merge
Choose a base branch
from

Conversation

marionbarker
Copy link
Contributor

Purpose:

Two of the headers in the Loop Report are printed as optionals, e.g.:

Optional(## WatchDataManager

Optional(## StatusExtensionDataManager

This messes up the syntax and makes it difficult to automatically parse the report.

Approach:

Add a force unwrap to the two lines of code.

Test:

With this change, all the markdown headers are now printed as expected, e.g.:

## WatchDataManager

## StatusExtensionDataManager

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.

1 participant