-
Notifications
You must be signed in to change notification settings - Fork 36
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
bump devops_tests to include checks for badge presence and Colab header cell #1403
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1403 +/- ##
=======================================
Coverage 84.90% 84.90%
=======================================
Files 375 375
Lines 9252 9252
=======================================
Hits 7855 7855
Misses 1397 1397 ☔ View full report in Codecov by Sentry. |
71b54ea
to
9f695b6
Compare
I think we should change devops_tests, because in probably every notebook we wrote View on Github instead of preview. So it will be easier to change test. Edit: I changed name of the badge in notebooks in PySDM as suggested in devops_tests PR#38 |
@slayoo, I don't know how to resolve this. I think it is due to different changes in the same files on your fork and PySDM repository. |
@AgnieszkaZaba, the conflicting file was renamed on the |
Thank you @AgnieszkaZaba! |
No description provided.