Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@sdavids
Copy link
Contributor

@sdavids sdavids commented Aug 24, 2024

  • updated all dependencies
  • migrated build.gradle to Kotlin-DSL
  • migrated settings.gradle to Kotlin-DSL
  • ensure reproducible build
  • minor fix to shell script

I suggest you review commit-by-commit …

sdavids added 17 commits August 24, 2024 20:30
Signed-off-by: Sebastian Davids <[email protected]>
Signed-off-by: Sebastian Davids <[email protected]>
Signed-off-by: Sebastian Davids <[email protected]>
Signed-off-by: Sebastian Davids <[email protected]>
Signed-off-by: Sebastian Davids <[email protected]>
Signed-off-by: Sebastian Davids <[email protected]>
Signed-off-by: Sebastian Davids <[email protected]>
Signed-off-by: Sebastian Davids <[email protected]>
Signed-off-by: Sebastian Davids <[email protected]>
@sdavids
Copy link
Contributor Author

sdavids commented Aug 24, 2024

I have not included the gradle-graal plugin due to it not being updated in 2 years.

It has not kept up with the changes in Gradle 8 nor the ones for the upcoming Gradle 9.

Also, there are several important open issues:

I really don't know why this plugin is so strict about the graalvm and jdk version, every single time Graal upgrades the plugin stops working.


The plugin seems to be abondoned.

@sdavids sdavids marked this pull request as ready for review August 24, 2024 21:03
@sdavids
Copy link
Contributor Author

sdavids commented Aug 24, 2024

@adoble adoble merged commit 90429ee into adoble:main Aug 27, 2024
@adoble
Copy link
Owner

adoble commented Aug 27, 2024

I agree with your appraisal of GraalVM and are happy to leave it out (I'll mention in the CHANGELOG that this is the case).

If anybody has a need for this, then please raise an issue.

@sdavids sdavids deleted the users/sdavids/feat/issue-55 branch August 27, 2024 15:06
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