-
Notifications
You must be signed in to change notification settings - Fork 327
[maven-tool] 对于默认的参数值,若为空,不添加该值 #355
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
Merged
CodeCasterX
merged 2 commits into
ModelEngine-Group:3.5.x
from
surpercodehang:enhancement-tools-json
Oct 30, 2025
Merged
[maven-tool] 对于默认的参数值,若为空,不添加该值 #355
CodeCasterX
merged 2 commits into
ModelEngine-Group:3.5.x
from
surpercodehang:enhancement-tools-json
Oct 30, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
4 tasks
loveTsong
reviewed
Oct 27, 2025
...l-maven-plugin/src/main/java/modelengine/fel/maven/complie/parser/ByteBuddySchemaParser.java
Outdated
Show resolved
Hide resolved
loveTsong
reviewed
Oct 27, 2025
...l-maven-plugin/src/main/java/modelengine/fel/maven/complie/parser/ByteBuddySchemaParser.java
Outdated
Show resolved
Hide resolved
7593d26 to
9d8fe80
Compare
9d8fe80 to
98a41ad
Compare
loveTsong
previously approved these changes
Oct 29, 2025
CodeCasterX
reviewed
Oct 29, 2025
...l-maven-plugin/src/main/java/modelengine/fel/maven/complie/parser/ByteBuddySchemaParser.java
Outdated
Show resolved
Hide resolved
ac06d69 to
9f33720
Compare
CodeCasterX
previously approved these changes
Oct 29, 2025
2494a5f to
78641ce
Compare
CodeCasterX
reviewed
Oct 29, 2025
framework/fit/java/fit-util/src/main/java/modelengine/fitframework/annotation/Property.java
Outdated
Show resolved
Hide resolved
78641ce to
93d1612
Compare
CodeCasterX
reviewed
Oct 29, 2025
...ava/services/tool-service/src/main/java/modelengine/fel/tool/support/MethodToolMetadata.java
Outdated
Show resolved
Hide resolved
93d1612 to
8e44b82
Compare
loveTsong
reviewed
Oct 30, 2025
...l-maven-plugin/src/main/java/modelengine/fel/maven/complie/parser/ByteBuddySchemaParser.java
Outdated
Show resolved
Hide resolved
8e44b82 to
4d1f60b
Compare
4d1f60b to
add7093
Compare
loveTsong
approved these changes
Oct 30, 2025
CodeCasterX
approved these changes
Oct 30, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔗 相关问题 / Related Issue
Issue 链接 / Issue Link: #352 👈👈
📋 变更类型 / Type of Change
📝 变更目的 / Purpose of the Change
java工具schema生成工具生成的default错误
用户注解未配置default时,生成默认值:default: ""
默认的数据不合理,即使是object类型也是default: ""
📋 主要变更 / Brief Changelog
🧪 验证变更 / Verifying this Change
测试步骤 / Test Steps
测试覆盖 / Test Coverage
📸 截图 / Screenshots
✅ 贡献者检查清单 / Contributor Checklist
请确保你的 Pull Request 符合以下要求 / Please ensure your Pull Request meets the following requirements:
基本要求 / Basic Requirements:
代码质量 / Code Quality:
测试要求 / Testing Requirements:
mvn -B clean package -Dmaven.test.skip=true/ Basic checks passmvn clean install/ Unit tests pass文档和兼容性 / Documentation and Compatibility:
📋 附加信息 / Additional Notes
审查者注意事项 / Reviewer Notes: