Skip to content

Update spec to reflect addition of Constraint and @Is#1159

Closed
gavinking wants to merge 3 commits into
jakartaee:mainfrom
gavinking:constraints-and-restrictions
Closed

Update spec to reflect addition of Constraint and @Is#1159
gavinking wants to merge 3 commits into
jakartaee:mainfrom
gavinking:constraints-and-restrictions

Conversation

@gavinking
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

@njr-11 njr-11 left a comment

Choose a reason for hiding this comment

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

Good idea to add this to the spec as well. I added a few comments. The ordering change seemed very minor although I couldn't figure out the reason for it, and I listed a possible ordering that I thought might make sense, but feel free to ignore that if you did have reasons for the other ordering.

Comment thread api/src/main/java/jakarta/data/repository/Query.java Outdated
Comment thread api/src/main/java/jakarta/data/repository/Find.java Outdated
Comment thread api/src/main/java/jakarta/data/repository/Find.java Outdated
Comment thread spec/src/main/asciidoc/repository.asciidoc Outdated
Comment thread spec/src/main/asciidoc/repository.asciidoc Outdated
Comment thread spec/src/main/asciidoc/repository.asciidoc Outdated
Comment thread spec/src/main/asciidoc/repository.asciidoc
Comment thread spec/src/main/asciidoc/repository.asciidoc
Co-authored-by: Nathan Rauh <nathan.rauh@us.ibm.com>
@gavinking
Copy link
Copy Markdown
Member Author

@njr-11 I've attempted to address your feedback. WDYT?

@gavinking gavinking force-pushed the constraints-and-restrictions branch from 4890254 to cec1f0d Compare June 25, 2025 19:08
@gavinking gavinking force-pushed the constraints-and-restrictions branch from cec1f0d to 5cc0693 Compare June 25, 2025 19:09
Copy link
Copy Markdown
Member

@njr-11 njr-11 left a comment

Choose a reason for hiding this comment

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

@njr-11 I've attempted to address your feedback. WDYT?

Looks good!

@njr-11 njr-11 changed the title Update spec to reflect addition of Constraint and @In Update spec to reflect addition of Constraint and @Is Jul 2, 2025
@otaviojava
Copy link
Copy Markdown
Contributor

@gavinking when you have time, could you rebase this one?

@gavinking
Copy link
Copy Markdown
Member Author

Superseded by #1170.

@gavinking gavinking closed this Jul 2, 2025
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.

3 participants