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
How to test new rules in real world projects easily #4408
Comments
Another way would be to fork the projects linked in the mentioned script and configure detekt in the Gradle scripts of those forks. |
That sounds good too. We could even ask to the maintainer if they are interested in detekt and merge it. |
I used https://github.com/cortinico/kotlin-android-template to test new rules or changes to the existing rule. |
That's a good playground. The problem is that it contains not much code so a codebase like https://github.com/mozilla-mobile/fenix would provide some real-world scenario. |
That's a good one! And they have detekt already in the project. It's not the last version but we can contribute to update it. |
I just open an issue in the Fenix repo to know if they are interested on detekt with type solving. Depending on they answer we can decide the next steps here. mozilla-mobile/fenix#23124 |
This issue is stale because it has been open 90 days with no activity. Please comment or this will be closed in 7 days. |
Related TWiStErRob/repros@d3019d7 |
Expected Behavior
It should be easy to test a new rule agains some Open Source sample projects to spot issues in the rule implementation.
Current Behavior
This is really difficult we have this script: https://github.com/detekt/detekt/blob/main/scripts/get_analysis_projects.kts but it is just usefull if you don't use Type Solving.
Context
#4033 and #4394 (review)
Possible solution
We should list projects that already use detekt and use those for this tests using the
--including-build
feature from gradle.The text was updated successfully, but these errors were encountered: