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

Use pluginManagement to include builds #6268

Merged
merged 1 commit into from Jul 14, 2023
Merged

Use pluginManagement to include builds #6268

merged 1 commit into from Jul 14, 2023

Conversation

3flex
Copy link
Member

@3flex 3flex commented Jul 8, 2023

This seems to be required to include detekt project in a composite build without errors.

includeBuild in pluginManagement was de-incubated in Gradle 8.

@detekt-ci detekt-ci added the build label Jul 8, 2023
This seems to be required to include detekt project in a composite
build without errors.

`includeBuild` in pluginManagement was de-incubated in Gradle 8.
@3flex 3flex added the housekeeping Marker for housekeeping tasks and refactorings label Jul 8, 2023
@3flex 3flex enabled auto-merge (squash) July 8, 2023 23:52
@codecov
Copy link

codecov bot commented Jul 8, 2023

Codecov Report

Merging #6268 (fe66939) into main (4411432) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #6268   +/-   ##
=========================================
  Coverage     85.20%   85.20%           
  Complexity     4035     4035           
=========================================
  Files           568      568           
  Lines         13448    13448           
  Branches       2378     2378           
=========================================
  Hits          11458    11458           
  Misses          831      831           
  Partials       1159     1159           

@3flex 3flex merged commit c9d120a into main Jul 14, 2023
23 checks passed
@3flex 3flex deleted the 3flex-patch-1 branch July 14, 2023 00:20
@cortinico
Copy link
Member

includeBuild in pluginManagement was de-incubated in Gradle 8.

Can we add a reference to this?
Also was this causing a build failure of some sort?

@detekt-ci
Copy link
Collaborator

Warnings
⚠️ This PR is approved with no milestone set. If merged, it won't appear in the detekt release notes.

Generated by 🚫 dangerJS against fe66939

@cortinico cortinico added this to the 2.0.0 milestone Jul 14, 2023
@3flex
Copy link
Member Author

3flex commented Jul 14, 2023

Can we add a reference to this?

gradle/gradle#24129

was this causing a build failure of some sort?

If detekt was included in another build, yes. The PR fixes that.

@TWiStErRob
Copy link
Member

This was a lifesaver to test #6331 (comment), thanks!

mgroth0 pushed a commit to mgroth0/detekt that referenced this pull request Feb 11, 2024
This seems to be required to include detekt project in a composite
build without errors.

`includeBuild` in pluginManagement was de-incubated in Gradle 8.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build housekeeping Marker for housekeeping tasks and refactorings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants