Skip to content

Conversation

jdpgrailsdev
Copy link
Contributor

@jdpgrailsdev jdpgrailsdev commented Mar 18, 2025

Modernizes the project build structure with the following changes:

  • Updates to latest Gradle release
  • Converts build scripts to Kotlin
  • Uses Airbyte Gradle plugin/removes duplicate build script logic
  • Address Checkstyle/PMD failures
  • Format code
  • Remove unused project files after conversion
  • Python 3.8 -> 3.9

I may need to do some testing of publishing as I am not 100% sure that I have matched the previous publishing configuration with regards to the GAV of the generated artifact.

Copy link
Contributor

@gosusnp gosusnp left a comment

Choose a reason for hiding this comment

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

LGTM, any reason to not convert this to kotlin?

@jdpgrailsdev
Copy link
Contributor Author

LGTM, any reason to not convert this to kotlin?

@gosusnp Nope. That's the next step. Didn't want to do it all in one go as it would make it hard to detect some of the changes.

@jdpgrailsdev
Copy link
Contributor Author

Confirmed that publish is working as it did previously.

@jdpgrailsdev jdpgrailsdev merged commit 7156fb4 into main Mar 18, 2025
6 checks passed
@jdpgrailsdev jdpgrailsdev deleted the jonathan/modernize-project branch March 18, 2025 17:53
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