Skip to content
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

feature/java-conventions-plugin-2 #3511

Merged
merged 50 commits into from
Jul 8, 2024
Merged

Conversation

rfc3092
Copy link
Contributor

@rfc3092 rfc3092 commented Jun 12, 2024

Bruker 3 convention plugins, for apps, libs og proxies. Har også en "intern" plugin for Sonar, mer om det nedenfor.

Det er tatt et bevisst valg på å holde dolly-[apps|libs|proxies] separate i stor grad, dvs. at noe dupliseres her. Tanken er a) å holde bruken av en plugin så enkel som mulig, b) ikke avskrekke noen fra å modifisere en plugin, og c) unngå problemer med at "interne" plugins må dra inn unødvendige egne plugins bare for å kunne sette config. F.eks. vil en "intern" plugin som brukes av både apps og libs måtte dra inn plugin "java", men libs bruker ikke "java" men "java-libs".

Det er heller ikke tanken at dependencies skal styres av konvensjoner, annet enn der det gir litt mening (Lombok, noe Spring-rammeverk for testing, krav fra NAIS mtp. logging/metrics etc. - altså dependencies som helt klart er listefyll). Det kan jo sikkert diskuteres på sikt.

Også verdt å merke seg at vi ikke kan lage en version catalog i Gradle vha. plugins. #hjertesukk

rfc3092 added 30 commits June 4, 2024 14:45
# Conflicts:
#	apps/budpro-service/build.gradle
@rfc3092 rfc3092 marked this pull request as ready for review June 18, 2024 11:25
@rfc3092 rfc3092 requested a review from a team as a code owner June 18, 2024 11:25
@rfc3092
Copy link
Contributor Author

rfc3092 commented Jun 18, 2024

Forenkle Gradle builds

Copy link
Contributor

@stigus stigus left a comment

Choose a reason for hiding this comment

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

Supert 🤩

Copy link
Contributor

@krharum krharum left a comment

Choose a reason for hiding this comment

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

Bra å få inn denne forenkling! 🥇

# Conflicts:
#	.github/workflows/app.dolly-backend.yml
@rfc3092 rfc3092 merged commit b632ce2 into master Jul 8, 2024
94 checks passed
@rfc3092 rfc3092 deleted the feature/java-conventions-plugin-2 branch July 8, 2024 07:46
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