-
Notifications
You must be signed in to change notification settings - Fork 11
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
Update actions to v3 to use node.js v16 and master
branch
#58
Conversation
Let's also see if the documentation builds correctly in this PR
Pull Request Test Coverage Report for Build 3307675532Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
Codecov ReportBase: 80.37% // Head: 79.90% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #58 +/- ##
==========================================
- Coverage 80.37% 79.90% -0.47%
==========================================
Files 8 8
Lines 214 214
==========================================
- Hits 172 171 -1
- Misses 42 43 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
OK, can replicate the git fetch failure |
This saves CO2 by preventing readme changes from rebuilding docs.
This is probably why docs did not deploy. Doh!
master
branch
With the fix in #61 I am pretty optimistic this will work now. |
It worked! At last we have docs-dev badge active. |
Tbh I don't know enough about GitHub actions as well. Let's use latest for now if it is working. It's unlikely that we use features that live on the bleeding edge, so latest should be equivalent to stable releases. |
OK, I'm testing it in a separate PR #62. Thanks! |
For the record, one can use |
Let's also see if the documentation builds correctly in this PR