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

update configuration-cache settings and change value to fail #27

Merged
merged 1 commit into from
May 30, 2024

Conversation

mona-apk
Copy link
Contributor

@mona-apk mona-apk commented May 27, 2024

Issue

Overview (Required)

  • Update configuration-cache settings and change the value of org.gradle.configuration-cache.problems to fail.
    • The current org.gradle.configuration-cache.problems settings use the notation from when it was an incubating feature.

Links

Screenshot (Optional if screenshot test is present or unrelated to UI)

Before After

@mona-apk mona-apk self-assigned this May 29, 2024
org.gradle.configuration-cache-problems=error
org.gradle.configuration-cache.problems=fail
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The org.gradle.configuration-cache.problems setting can be either fail or warn. The original value was error, and it has now been updated to fail.

@mona-apk mona-apk marked this pull request as ready for review May 29, 2024 13:42
@mona-apk mona-apk changed the title WIP: update configuration-cache settings and change value to fail update configuration-cache settings and change value to fail May 29, 2024
Copy link
Contributor

@RyuNen344 RyuNen344 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mona-apk mona-apk merged commit d2861dc into main May 30, 2024
@mona-apk mona-apk deleted the feature/update-configuration-cache-settings branch May 30, 2024 08:12
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.

2 participants