-
Notifications
You must be signed in to change notification settings - Fork 2
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: Add option to set a Nextcloud target version or parse the appinfo #164
Conversation
bc528cb
to
54ad406
Compare
Conflicts |
54ad406
to
2315a46
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #164 +/- ##
==========================================
+ Coverage 77.19% 86.23% +9.04%
==========================================
Files 5 6 +1
Lines 57 109 +52
==========================================
+ Hits 44 94 +50
- Misses 13 15 +2 ☔ View full report in Codecov by Sentry. |
@skjnldsv resolved and added unit tests for 100% coverage of the added utils script |
Should we enable it by default on the global config? |
I think this would be useful, what do you think @ChristophWurst ? (As you requested that feature in the first place) |
You can set a `targetVersion` option like `targetVersion: '25.0.0'` to only report deprecations / removals that happend before that version. Moreover you can also set `parseAppInfo: true` to parse the `appinfo/info.xml` from a Nextcloud app and only report deprecations / removals before the `max-version` of the Nextcloud dependency. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
2315a46
to
e780f8a
Compare
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Inspired by #1, but I think parsing the appinfo is more helpful.
Summary
Allow to set a
targetVersion
option liketargetVersion: '25.0.0'
to only report deprecations / removals that happend before that version.Moreover you can also set
parseAppInfo: true
to parse theappinfo/info.xml
from a Nextcloud app and only report deprecations / removals before themax-version
of the Nextcloud dependency.