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

Misleading --debug suggestion #4857

Closed
TWiStErRob opened this issue May 23, 2022 · 4 comments
Closed

Misleading --debug suggestion #4857

TWiStErRob opened this issue May 23, 2022 · 4 comments

Comments

@TWiStErRob
Copy link
Member

Expected Behavior

The BindingContext was created with 1 issues. Run detekt CLI with --debug or set `detekt { debug = true }` in Gradle to see the error messages.

or

The BindingContext was created with 1 issues. Run detekt with debug flag to see the error messages.

or similar

Current Behavior

The BindingContext was created with 1 issues. Run detekt with --debug to see the error messages.

Context

Just saw a warning and wanted to get more info, after 10 minutes of digging into gradlew --debug, I realized I'm not getting more info unless I turn on DetektExtension.debug. I opened this to start a discussion and help others in the future.

@cortinico
Copy link
Member

I realized I'm not getting more info unless I turn on DetektExtension.debug. I opened this to start a discussion and help others in the future.

Good point. This warning is fired from detekt-core which is "integration agnostic". I think --debug was a wrong flag decision in the first place. Maybe we could link to a "debug mode" webpage?

@TWiStErRob
Copy link
Member Author

Yeah, that's perfect, a separately maintainable and improvable page. We could even explain some common issues like this one, linking to specific issues, etc.

@TWiStErRob
Copy link
Member Author

Does #5300 fix this? I would like still to see a website for explaining debugging and common issues, but #5300 implemented my initial suggestion from this just GitHub didn't pick it up.

@BraisGabin
Copy link
Member

Yes it does. If you see a lack of documentation, please open a new issue.

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

3 participants