-
Notifications
You must be signed in to change notification settings - Fork 446
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
feat(linter): show missing libraries corresponding package #4430
Conversation
997881f
to
3ed2dc1
Compare
3ed2dc1
to
bac2a0d
Compare
@@ -2,6 +2,6 @@ Running linters... | |||
Running linter: classic | |||
Running linter: library | |||
Lint warnings: | |||
- library: linter-test: missing dependency 'libcaca.so.0'. (https://snapcraft.io/docs/linters-library) | |||
- library: linter-test: missing dependency 'libslang.so.2'. (https://snapcraft.io/docs/linters-library) | |||
- library: linter-test: missing dependency 'libcaca.so.0'. (provided by 'libcaca0') (https://snapcraft.io/docs/linters-library) |
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 am not sure it will be easy to correlate provided by with a possible stage-packages entry. Should be fine if the documentation mentions that.
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.
Looks good, thanks!
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## main #4430 +/- ##
==========================================
- Coverage 89.20% 89.18% -0.03%
==========================================
Files 322 321 -1
Lines 21668 21650 -18
==========================================
- Hits 19329 19308 -21
- Misses 2339 2342 +3
... and 5 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Show missing libraries that belong to any system packages when possible.
Based on @sergio-costas PR #4293