Skip to content

Conversation

@alex-amenos
Copy link
Owner

⚡️ Proposed Changes

Properties should be assigned using the 'propName = value' syntax. Setting a property via the Gradle-generated 'propName value' or 'propName(value)' syntax in Groovy DSL has been deprecated

ℹ️ Additional Info

  • Add any additional useful context or info

🔗 Related Links

  • Add helpful links for this pull request

✅ Checklist

  • Unit Tests
  • Integration Tests
  • Compose Tests
  • Screenshot Tests
  • Updated string
  • Manually tested

📷 Screenshots

…propName = value' syntax.

Setting a property via the Gradle-generated 'propName value' or 'propName(value)' syntax in Groovy DSL has been deprecated
@alex-amenos alex-amenos self-assigned this Nov 19, 2025
Copilot AI review requested due to automatic review settings November 19, 2025 22:48
@alex-amenos alex-amenos added the WIP Work In Progres label Nov 19, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes Gradle build scripts by migrating from the deprecated Groovy DSL property assignment syntax (propName value) to the recommended syntax (propName = value). This refactoring addresses Gradle deprecation warnings and aligns with current best practices.

Key Changes:

  • Updated property assignments across all build.gradle files to use the = operator
  • Migrated fileMode to the modern filePermissions DSL block
  • Standardized method call syntax with explicit parentheses where appropriate

Reviewed Changes

Copilot reviewed 19 out of 20 changed files in this pull request and generated no comments.

Show a summary per file
File Description
shared/testing/build.gradle Updated namespace property assignment
shared/kotlin/build.gradle Updated namespace property assignment
shared/core/build.gradle Updated namespace property assignment
shared/api/build.gradle Updated namespace property assignment
settings.gradle Updated maven repository URL assignments
feature/settings/build.gradle Updated resourcePrefix and namespace assignments
feature/root/build.gradle Updated resourcePrefix and namespace assignments
feature/posts/build.gradle Updated resourcePrefix and namespace assignments
feature/my-playground/build.gradle Updated resourcePrefix and namespace assignments
feature/location-tracker/build.gradle Updated resourcePrefix and namespace assignments
feature/home/build.gradle Updated resourcePrefix and namespace assignments
feature/game/ballclicker/build.gradle Updated resourcePrefix assignment
feature/file-downloader/build.gradle Updated resourcePrefix and namespace assignments
feature/authentication/build.gradle Updated resourcePrefix and namespace assignments
buildSystem/gradle/common-app-signing.gradle Updated signing config and build type properties
buildSystem/gradle/common-app-base.gradle Updated SDK versions, build types, and lint configuration properties
buildSystem/gradle/common-android-compose.gradle Updated compose buildFeature property
buildSystem/gradle/common-android-base.gradle Updated SDK versions, build types, buildConfig, and lint properties
build.gradle Updated test logging properties and file permission configuration
app/build.gradle Updated namespace property assignment

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@alex-amenos alex-amenos removed the WIP Work In Progres label Nov 23, 2025
@alex-amenos alex-amenos merged commit 04d3923 into main Nov 23, 2025
1 check passed
@alex-amenos alex-amenos deleted the refactor/gradle branch November 23, 2025 21:26
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.

1 participant