Skip to content

feat: add fluent header api for no-bean mode#943

Merged
psxjoy merged 2 commits into
apache:mainfrom
bengbengbalabalabeng:feat-issue-941
Jul 8, 2026
Merged

feat: add fluent header api for no-bean mode#943
psxjoy merged 2 commits into
apache:mainfrom
bengbengbalabalabeng:feat-issue-941

Conversation

@bengbengbalabalabeng

Copy link
Copy Markdown
Contributor

Purpose of the pull request

Closed: #941

What's changed?

  • Added the HeadBuilder as a builder helper.
  • Added the overloaded method head(Consumer<HeadBuilder>) to complement the use of head(List<List<String>>).

Usage Example: #941

Checklist

  • I have read the Contributor Guide.
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

public abstract class AbstractParameterBuilder<T extends AbstractParameterBuilder, C extends BasicParameter> {
/**
* You can only choose one of the {@link #head(List)} and {@link #head(Class)}
* You can only choose one of the ({@link #head(List)} or {@link #head(Consumer)}) and {@link #head(Class)}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Since this remark should be modify,better not using 'You' .

既然这个注释已经修改了,我觉得原始的注释有点太口语化了,最好不要用“你”这种词汇?

}

/**
* You can only choose one of the ({@link #head(List)} or {@link #head(Consumer)}) and {@link #head(Class)}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

same above


/**
* You can only choose one of the {@link #head(List)} and {@link #head(Class)}
* You can only choose one of the ({@link #head(List)} or {@link #head(Consumer)}) and {@link #head(Class)}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

same above

@psxjoy psxjoy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@psxjoy psxjoy merged commit d7fae5a into apache:main Jul 8, 2026
9 checks passed
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.

[Enhancement] Add Fluent Header API for No-Bean Mode

2 participants