Skip to content

Fix Code Scanning warnings#939

Merged
MGaetan89 merged 1 commit intomainfrom
fix_code_scanning_warnings
Mar 11, 2025
Merged

Fix Code Scanning warnings#939
MGaetan89 merged 1 commit intomainfrom
fix_code_scanning_warnings

Conversation

@MGaetan89
Copy link
Contributor

Pull request

Description

This commit fixes Code Scanning warnings, in particular:

Changes made

  • Use Android KTX extensions where possible.
  • Address some Dependency Analysis warnings.

Checklist

  • APIs have been properly documented (if relevant).
  • The documentation has been updated (if relevant).
  • New unit tests have been written (if relevant).
  • The demo has been updated (if relevant).

@MGaetan89 MGaetan89 self-assigned this Mar 11, 2025
@github-project-automation github-project-automation bot moved this to ✏️ Draft in Pillarbox Mar 11, 2025
@MGaetan89 MGaetan89 moved this from ✏️ Draft to 🍿 Code Review in Pillarbox Mar 11, 2025
@github-actions
Copy link

Code Coverage

Overall Project 51.09% 🟢
Files changed 100% 🟢

Module Coverage
:pillarbox-core-business 82.17% 🟢
Files
Module File Coverage
:pillarbox-core-business ChapterAdapter.kt 100% 🟢
IlUrl.kt 95.98% 🟢
SRGAssetLoader.kt 90.73% 🟢

@MGaetan89 MGaetan89 requested a review from StaehliJ March 11, 2025 13:37
@MGaetan89 MGaetan89 added this pull request to the merge queue Mar 11, 2025
Merged via the queue into main with commit 63e62ea Mar 11, 2025
10 checks passed
@MGaetan89 MGaetan89 deleted the fix_code_scanning_warnings branch March 11, 2025 18:27
@github-project-automation github-project-automation bot moved this from 🍿 Code Review to ✅ Done in Pillarbox Mar 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants