Skip to content

Migration to Jakarta EL 4 compatible version of JUEL - #915

Open
dmaidaniuk wants to merge 5 commits into
HubSpot:masterfrom
dmaidaniuk:feature/jakarta-el-4
Open

Migration to Jakarta EL 4 compatible version of JUEL#915
dmaidaniuk wants to merge 5 commits into
HubSpot:masterfrom
dmaidaniuk:feature/jakarta-el-4

Conversation

@dmaidaniuk

Copy link
Copy Markdown

PR for issue #809

Please note, that here used a fork of JUEL library upgraded to Jakarta EL 4:

https://github.com/UkrSoftTech/juel/tree/jakarta-el-4

@dmaidaniuk

Copy link
Copy Markdown
Author

The original repo of JUEL https://github.com/beckchr/juel is not maintained since 2018. So, either the forked repo should be deployed to the Sonatype repository with a changed group id, or sources of JUEL should be included in the Jinjava library.

@dmaidaniuk

Copy link
Copy Markdown
Author

@jasmith-hs please take a look.

@jasmith-hs

Copy link
Copy Markdown
Contributor

We will likely lean more towards including the sources into the Jinjava library like in #810
Since the original repo isn't maintained. It will be simpler to fix any bugs this way/upgrade to EL 5 if we desire that.

We could also maintain our own fork of JUEL cc @mattcoley @boulter

@dmaidaniuk

Copy link
Copy Markdown
Author

@jasmith-hs I can transfer ownership of https://github.com/UkrSoftTech/juel/ repository to HubSpot organization. Also, in that fork I made the migration to EL 5 on another branch https://github.com/UkrSoftTech/juel/tree/jakarta-el-5.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants