Skip to content

Remove redundant ExecutionGraphQlRequest interface declaration #1218

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

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

Conversation

trinhdvt
Copy link

What

Removes redundant implements ExecutionGraphQlRequest from WebGraphQlRequest class declaration.

Why

WebGraphQlRequest extends DefaultExecutionGraphQlRequest, which already implements ExecutionGraphQlRequest. The explicit interface declaration was therefore redundant and has been removed to clean up the code.

WebGraphQlRequest already implements ExecutionGraphQlRequest through its
parent class DefaultExecutionGraphQlRequest, making the explicit interface
declaration redundant.

Signed-off-by: do.vantrinh <do.vantrinh@moneyforward.co.jp>
@trinhdvt trinhdvt force-pushed the Remove-redundant-interface-declaration-in-WebGraphQlRequest branch from ceb7446 to 2838b41 Compare May 24, 2025 08:25
@trinhdvt trinhdvt marked this pull request as ready for review May 24, 2025 08:26
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 24, 2025
@bclozel bclozel added type: task A general task in: web Issues related to web handling and removed status: waiting-for-triage An issue we've not yet triaged labels May 24, 2025
@bclozel bclozel added this to the 1.4.1 milestone May 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: web Issues related to web handling type: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants