-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Issue#322 add sofa plugin #384
Conversation
Codecov Report
@@ Coverage Diff @@
## master #384 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 23 24 +1
Lines 61 62 +1
Branches 1 1
======================================
- Misses 61 62 +1
Continue to review full report at Codecov.
|
* @return the plugin data handler | ||
*/ | ||
@Bean | ||
public PluginDataHandler sofaoPluginDataHandler() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sofaoPluginDataHandler
这里是不是多了个o
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
嗯 我在下次合的时候改掉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这次pr一起改了把,改了直接提交就好
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 另外我想问下,license header是不是每个文件都要加的,我看现在有些文件有,有些没有 @yu199195
昨天太晚了我没加 后续我都会补上的 |
@tydhot 在这个pr里面把文件头山的 lincense都加上把 |
@yu199195 license header如果是必须的就maven中增加检查头部吧? |
* [apache#322]prepare add sofa plugin * [apache#322]add sofa plugin * [apache#322]add licenses
No description provided.