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

Detekt - Enable Type Resolution (detektMain/Test) #17011

Open
Tracked by #16892
ParaskP7 opened this issue Aug 5, 2022 · 3 comments
Open
Tracked by #16892

Detekt - Enable Type Resolution (detektMain/Test) #17011

ParaskP7 opened this issue Aug 5, 2022 · 3 comments
Assignees
Milestone

Comments

@ParaskP7
Copy link
Contributor

ParaskP7 commented Aug 5, 2022

Parent #16892

This issue is about enabling type resolution for Detekt via detektMain/Test tasks.

TL;DR: We might want to start using detektMain/Test in favor of detekt task as it is nowadays the recommend way to configure Detekt (see Using Type Resolution and more specifically here and here). The plain detekt task is not running with type resolution enabled, while the detektMain/Test tasks are (for more info watch this nice Detekt related talk here, go to 30:30, up until 34:00).

@ParaskP7 ParaskP7 added this to the Future milestone Aug 5, 2022
@ParaskP7 ParaskP7 self-assigned this Aug 5, 2022
@peril-wordpress-mobile
Copy link

Fails
🚫

Please add a type label to this issue. e.g. '[Type] Enhancement'

Generated by 🚫 dangerJS

2 similar comments
@peril-wordpress-mobile
Copy link

Fails
🚫

Please add a type label to this issue. e.g. '[Type] Enhancement'

Generated by 🚫 dangerJS

@peril-wordpress-mobile
Copy link

Fails
🚫

Please add a type label to this issue. e.g. '[Type] Enhancement'

Generated by 🚫 dangerJS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant