Skip to content

Fix build scan warning about build cache configuration #5690

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

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

3flex
Copy link
Member

@3flex 3flex commented Jan 11, 2023

Message: "The build cache configuration of the root build differs from the build cache configuration of the early evaluated ':build-logic', ':detekt-gradle-plugin' included builds. It is recommended to keep them consistent."

See example here: https://ge.detekt.dev/s/pvz4ogayjxfzc/performance/build-cache

In theory this can be fixed by copying the buildCache {} config from the root settings file to the build-logic and detekt-gradle-plugin settings files, but this doesn't seem to be working (see gradle/gradle#21630).

In the meantime, have used this reference as a way to correct it:

Gradle’s composite build feature allows including other complete Gradle builds into another. Such included builds will inherit the build cache configuration from the top level build, regardless of whether the included builds define build cache configuration themselves or not.

Using pluginManagement reverses the order of evaluation which leads to the warning due to the bug mentioned above, so switching to the other, stable way to add includeBuild until the Gradle bug is fixed.

Message: The build cache configuration of the root build differs from the
build cache configuration of the early evaluated ':build-logic',
':detekt-gradle-plugin' included builds. It is recommended to keep them
consistent.
@github-actions github-actions bot added the build label Jan 11, 2023
@codecov
Copy link

codecov bot commented Jan 11, 2023

Codecov Report

Merging #5690 (3260ea3) into main (465922f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #5690   +/-   ##
=========================================
  Coverage     84.46%   84.46%           
  Complexity     3733     3733           
=========================================
  Files           543      543           
  Lines         12701    12701           
  Branches       2221     2221           
=========================================
  Hits          10728    10728           
  Misses          865      865           
  Partials       1108     1108           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

Successfully merging this pull request may close these issues.

2 participants