Skip to content

Gradle: clean up dependency and plugin version management#3970

Merged
maniac103 merged 1 commit intoopenhab:mainfrom
MartinHorsfield:gradle-cleanup
Feb 6, 2026
Merged

Gradle: clean up dependency and plugin version management#3970
maniac103 merged 1 commit intoopenhab:mainfrom
MartinHorsfield:gradle-cleanup

Conversation

@MartinHorsfield
Copy link
Copy Markdown
Contributor

Summary

This PR cleans up and simplifies the Gradle configuration by centralizing dependency and plugin versions using a Gradle Version Catalog. The goal is to make the build files easier to read and maintain without changing any behavior.

What changed

I consolidated dependency and plugin versions that were previously spread across multiple Gradle files into a single libs.versions.toml file. This reduces duplication and makes it clearer where versions are defined. I also did some light formatting and organization of the Gradle files to improve readability.

What did not change

There are no dependency upgrades or downgrades in this PR. All versions remain exactly the same as before. There are no functional or runtime changes, and there is no impact to the full or foss build variants.

Motivation

While getting familiar with the project and the build setup, I found the Gradle configuration a bit difficult to follow as a new contributor. This change is meant to make the project more approachable and easier to maintain going forward.

Testing

The project syncs successfully in Android Studio. I ran clean and assemble builds locally and verified that both full and foss variants still resolve and build as expected.

Notes

This is my first contribution to the project. Feedback is very welcome and I am happy to adjust anything to better match the project’s conventions.

@MartinHorsfield MartinHorsfield force-pushed the gradle-cleanup branch 5 times, most recently from aba6760 to 3347868 Compare February 3, 2026 05:54
Copy link
Copy Markdown
Contributor

@maniac103 maniac103 left a comment

Choose a reason for hiding this comment

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

LGTM, except something went wrong during rebasing: commit title is off and the changes in strings.xml shouldn't be part of this PR.

@MartinHorsfield
Copy link
Copy Markdown
Contributor Author

LGTM, except something went wrong during rebasing: commit title is off and the changes in strings.xml shouldn't be part of this PR.

Thanks for the quick review. Not sure how I didn't notice that!

Fixed now

@maniac103
Copy link
Copy Markdown
Contributor

Commit title still seems off?

@MartinHorsfield MartinHorsfield changed the title gradle file organization Gradle: clean up dependency and plugin version management Feb 3, 2026
@MartinHorsfield
Copy link
Copy Markdown
Contributor Author

Commit title still seems off?

Is this a better commit title?

@maniac103
Copy link
Copy Markdown
Contributor

Yes, but I actually was referring to the commit title (see the commits page), not the PR title 😉

@MartinHorsfield
Copy link
Copy Markdown
Contributor Author

Yes, but I actually was referring to the commit title (see the commits page), not the PR title 😉
Ahh! should be correct now

Copy link
Copy Markdown
Contributor

@maniac103 maniac103 left a comment

Choose a reason for hiding this comment

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

@mueller-ma Any concerns?

@maniac103 maniac103 merged commit 6127b93 into openhab:main Feb 6, 2026
8 checks passed
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