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

doc: Update readme to include link to reference implementation. #235

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bjorndown
Copy link

@bjorndown bjorndown commented Jun 27, 2024

I found this information too hard to find.

@markt-asf
Copy link
Contributor

There is no such thing as a reference implementation. There are compatible implementations which depend on version. It isn't always Expressly (although I expect Expressly with pass the 6.0 TCK soon).

Compatible implementations can be found via https://jakarta.ee/specifications/expression-language/
(each version has its own set of CIs).

I think an update to the readme to link to https://jakarta.ee/specifications/expression-language/ would be reasonable.

@bjorndown
Copy link
Author

bjorndown commented Jun 27, 2024

There is no such thing as a reference implementation

Oh ok that is interesting because here it says reference implementation: https://docs.jboss.org/hibernate/stable/validator/reference/en-US/html_single/#validator-gettingstarted-uel

But I noticed that they link to https://github.com/eclipse-ee4j/el-ri which redirects to this repo here. I guess somebody only updated the maven dependency instead of rephrasing the paragraph.

I am fine with linking to https://jakarta.ee/specifications/expression-language/ because that makes it easier for people to find an implementation.

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.

None yet

2 participants