Skip to content
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

Upgrade detekt and ktlint to take advantage of the new features of these libraries. #4232

Open
MohitMaliFtechiz opened this issue Feb 19, 2025 · 0 comments · May be fixed by #4233
Open

Upgrade detekt and ktlint to take advantage of the new features of these libraries. #4232

MohitMaliFtechiz opened this issue Feb 19, 2025 · 0 comments · May be fixed by #4233
Assignees
Milestone

Comments

@MohitMaliFtechiz
Copy link
Collaborator

Describe the bug
We are adding support for Android 16 in #4231. As part of this, we have upgraded our Gradle and dependencies.

After upgrading Gradle, the new lint checks bring significant improvements. Additionally, our project includes detekt and ktlint, so we should upgrade them as well to leverage their enhancements for better code quality and robustness.

However, upgrading these libraries in #4231 would make the process more complex, as it requires extensive refactoring. Therefore, it is better to upgrade them separately.

Environment

  • Version of Kiwix Android: 3.14.0
@MohitMaliFtechiz MohitMaliFtechiz added this to the 3.15.0 milestone Feb 19, 2025
@MohitMaliFtechiz MohitMaliFtechiz self-assigned this Feb 19, 2025
@MohitMaliFtechiz MohitMaliFtechiz changed the title Upgraded detekt and ktlint version to take advantage of new feature of these libraries. Upgrade detekt and ktlint to take advantage of the new features of these libraries. Feb 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant