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

Add code coverage #352

Merged
merged 4 commits into from Jul 3, 2019
Merged

Add code coverage #352

merged 4 commits into from Jul 3, 2019

Conversation

MihaZupan
Copy link
Collaborator

Resolves #351

e6dd2cf simply removes some preprocessor directives as all build targets now support unsafe and fixed strings.

@xoofx xoofx merged commit 3106a49 into xoofx:master Jul 3, 2019
@xoofx
Copy link
Owner

xoofx commented Jul 3, 2019

I'm going to update the appveyor to perform the upload to coveralls

@xoofx
Copy link
Owner

xoofx commented Jul 3, 2019

Thanks!

@xoofx
Copy link
Owner

xoofx commented Jul 3, 2019

I have added the badge and it now uploads to markdig results coverall.io
I was not expecting the code to be at 95%... it's a good surprise!

@MihaZupan
Copy link
Collaborator Author

And a big chunk of those 5 is not having the prefix tree tests here. Markdig tests are great.

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.

Code coverage
2 participants