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

prepare switch over to kotlin, "new" java date formatting #5195

Merged
merged 7 commits into from
Dec 23, 2023

Conversation

soloturn
Copy link
Contributor

gradle groovy allows quite similar syntax for groovy and kotlin, do a first step to switch over to kotlin by replacing quotes, use java date formatting introduced in java-8, and register some tasks so they do not get configured when not executed.

@soloturn soloturn force-pushed the qa/prep-kotlin branch 2 times, most recently from 8ef5bd2 to a5352e5 Compare December 17, 2023 12:06
@soloturn soloturn force-pushed the qa/prep-kotlin branch 2 times, most recently from 5ebb8c7 to 7f0542d Compare December 17, 2023 12:59
Copy link
Contributor

@BenjaminAmos BenjaminAmos left a comment

Choose a reason for hiding this comment

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

These changes appear fairly self-explanatory.

One thing to be aware of is that double quotes in Groovy are actually GString types, rather than String. Single quotes are string literals ("real" String instances). Groovy usually converts seamlessles between the two though, so it should be fine.

@soloturn soloturn merged commit bf78a58 into develop Dec 23, 2023
9 checks passed
@soloturn soloturn deleted the qa/prep-kotlin branch December 23, 2023 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants