-
Notifications
You must be signed in to change notification settings - Fork 6
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
Implemented import dashboards from a directory #26
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #26 +/- ##
=======================================
Coverage ? 19.73%
=======================================
Files ? 5
Lines ? 446
Branches ? 0
=======================================
Hits ? 88
Misses ? 358
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…oolbox#24) Also, satisfy CI due to an unrelated error. --------- Co-authored-by: Vitalii_Rymar <[email protected]>
Also thanks for this excellent improvement, Vitalii. Can you also reformat the code to satisfy CI, like you did with your other patch? |
Will do tomorrow |
…feture-import-dir-support
…/grafana-import into feture-import-dir-support
Hi @amotl ! Not sure why export tests are failing, as this functionality was not touched. Checking... |
Maybe you need to rebase your branch on top of main? If you can't manage to make CI succeed, I can also have a look later to support you. |
Closing this PR and creating a new one rebased on upstream/main |
Solution for issue #25.