-
Notifications
You must be signed in to change notification settings - Fork 29
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
refactor(buid): apply the edc build plugin and the version catalog #58
refactor(buid): apply the edc build plugin and the version catalog #58
Conversation
98df0a1
to
c001b8f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
love the 🔪
c001b8f
to
9254f17
Compare
settings.gradle.kts
Outdated
versionCatalogs { | ||
create("libs") { | ||
from("org.eclipse.edc:edc-versions:0.0.1-SNAPSHOT") | ||
version("picocli", "4.6.3") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not possible to have these lib versions in the gradle.properties file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried on top level of the file and it didn't work. Checking it now probably it would work inside the single block
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In case i will raise an issue for fixing this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I though this was merged, it's still opened so i will change it here
9254f17
to
1a6753b
Compare
What this PR changes/adds
Apply the edc build plugin and the version catalog
Why it does that
simplify build file and consistency with EDC
Linked Issue(s)
Closes #56
Checklist
no-changelog
)