-
Notifications
You must be signed in to change notification settings - Fork 45
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
Provide the outcome of integration and documentation tests as downloadable artifacts from each Cirrus CI test page #384
Conversation
Codecov Report
@@ Coverage Diff @@
## master #384 +/- ##
=======================================
Coverage 94.63% 94.63%
=======================================
Files 8 8
Lines 839 839
=======================================
Hits 794 794
Misses 45 45 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I looked into the readthedocs settings and docuemtnation, but I can't understand why we don't have the automatic build (the settings are as they should be).
I'm totally happy with this solution! LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes look great to me as well!
🚀 PR was released in |
Closes Nothing
Proposed Changes
Change Type
bugfix
(+0.0.1)minor
(+0.1.0)major
(+1.0.0)refactoring
(no version update)test
(no version update)infrastructure
(no version update)documentation
(no version update)other
Checklist before review