Skip to content

Fix Gradle deprecated property assignment warnings in build files#5633

Open
kedarvenkateshkumar wants to merge 1 commit intoapache:developfrom
kedarvenkateshkumar:fix/gradle-deprecation-warnings
Open

Fix Gradle deprecated property assignment warnings in build files#5633
kedarvenkateshkumar wants to merge 1 commit intoapache:developfrom
kedarvenkateshkumar:fix/gradle-deprecation-warnings

Conversation

@kedarvenkateshkumar
Copy link

Summary

Fixed Gradle deprecated property assignment syntax warnings
in multiple build.gradle files across the project.

Changes

  • build.gradle: Fixed propName value → propName = value syntax
  • fineract-doc/build.gradle: Fixed same syntax warnings
  • fineract-war/build.gradle: Fixed compression, enabled properties
  • integration-tests/build.gradle: Fixed containerId property
  • oauth2-tests/build.gradle: Fixed containerId property

Testing

Command: gradlew.bat compileJava --warning-mode all
Result: BUILD SUCCESSFUL - warnings resolved

@airajena
Copy link
Contributor

  • Access Apache Fineract Jira and assign yourself the ticket
  • Comment that you are working on it
  • Request feedback
  • Fork the Apache Fineract repository
  • Do the proper changes
  • Run the Spotless Apply (it is possible that it should be run in different projects/modules)
  • Build in your local dev environment and confirm that there are no issues
  • Sign the commit and set the comment using the "FINERACT-{NUMBER}: {Jira text title}"
  • Create the Pull request, make sure to name the PR "FINERACT-{NUMBER}: {Jira text title}" (it is verified as part of the GitHub Actions)
  • Please be patient, there are several PRs at this moment and it is possible that maintainer will request to do rebase

Copy link
Contributor

@IOhacker IOhacker left a comment

Choose a reason for hiding this comment

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

Review the comments

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.

3 participants