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

Remove deprecated search limit option from code base #1467

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

stmaute
Copy link
Contributor

@stmaute stmaute commented Aug 31, 2022

With this PR the deprecated limit option and all dependent code, documentation and API docs are removed.

Fixes #1459

Stefan added 4 commits August 31, 2022 11:56
remove use of limit option in search;
minor code adjustments;

Signed-off-by: Stefan <[email protected]>
Signed-off-by: Stefan Maute <[email protected]>
remove limit and skip vom Query and QueryBuilder interface;
add exclusions for japicmp-maven-plugin;
adapted unit tests;

Signed-off-by: Stefan Maute <[email protected]>
@stmaute stmaute added this to the 4.0.0 milestone Aug 31, 2022
@thjaeckle thjaeckle changed the title Remove deprecated limit option from code base Remove deprecated search limit option from code base Aug 31, 2022
@thjaeckle thjaeckle marked this pull request as draft September 9, 2022 13:26
@thjaeckle thjaeckle added the on hold Waiting for feedback / requested changes on an issue/PR label Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
on hold Waiting for feedback / requested changes on an issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove deprecated limit option from search API
2 participants