-
Notifications
You must be signed in to change notification settings - Fork 838
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
ci: add test to ensure plugins support latest module versions #670
ci: add test to ensure plugins support latest module versions #670
Conversation
Codecov Report
@@ Coverage Diff @@
## master #670 +/- ##
==========================================
- Coverage 91.47% 91.41% -0.06%
==========================================
Files 216 215 -1
Lines 10048 10010 -38
Branches 913 910 -3
==========================================
- Hits 9191 9151 -40
- Misses 857 859 +2
|
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.
LGTM, but is there a way to get alerted on that ? Something directly on gitter or an issue on github would be easier no ?
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.
LGTM
@dyladan our circle ci config has changed since then, is this yet needed ? |
Think something is still needed, but the ci infra is completely changed and there was no way to report results. Will close this until we can come up with a better solution |
…mamir) open-telemetry#621 (open-telemetry#670) Co-Authored-By: Amir Blum [email protected]
Which problem is this PR solving?