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

Fix detekt source override dropping all source #5049

Merged
merged 12 commits into from
Nov 6, 2024
Merged

Fix detekt source override dropping all source #5049

merged 12 commits into from
Nov 6, 2024

Conversation

rli
Copy link
Contributor

@rli rli commented Nov 5, 2024

In CI, the override just drops all the source because all paths contain /codebuild

> Task :plugin-amazonq:codewhisperer:jetbrains-community:detektMain NO-SOURCE
Skipping task ':plugin-amazonq:codewhisperer:jetbrains-community:detektMain' as it has no source files and no previous output files.
Resolve mutations for :plugin-core:jetbrains-community:compileTestFixturesJava (Thread[#144,Execution worker Thread 4,5,main]) started.
:plugin-core:jetbrains-community:compileTestFixturesJava (Thread[#144,Execution worker Thread 4,5,main]) started.

License

I confirm that my contribution is made under the terms of the Apache 2.0 license.

@rli rli requested a review from a team as a code owner November 5, 2024 18:02
Copy link

github-actions bot commented Nov 5, 2024

Qodana Community for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

@rli rli changed the title Remove detekt source override Fix detekt source override dropping all source Nov 5, 2024
@rli rli requested a review from a team as a code owner November 6, 2024 00:00
@rli rli merged commit 5f74b1e into main Nov 6, 2024
15 of 17 checks passed
@rli rli deleted the rli/fix-detekt branch November 6, 2024 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants