Skip to content

Conversation

@djarnis73
Copy link
Contributor

Resolves a build warning due to incompatible versions of slf4j.

Change in HandlebarsTemplateEngine is due to a change in handlebars where they no longer shade commons text in their jar.

This breaks the licens check, but I don't know the implications.

Resolves a build warning due to incompatible versions of slf4j.

Change in HandlebarsTemplateEngine is due to a change in handlebars where they no longer shade commons text in their jar.

Exclude the nashhorn-core dependency from handlebars since it is not license compliant. It is only needed for javascript support in templates which we do not use.
@djarnis73 djarnis73 force-pushed the bump-handlebars-dependency branch from c759511 to 096de5d Compare November 3, 2025 09:16
@slinkydeveloper slinkydeveloper self-requested a review November 3, 2025 14:02
Copy link
Contributor

@slinkydeveloper slinkydeveloper left a comment

Choose a reason for hiding this comment

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

Looks good, let's see if CI passes, thanks for the contrib! :)

@slinkydeveloper slinkydeveloper merged commit fa849c2 into restatedev:main Nov 3, 2025
6 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants