Skip to content
This repository has been archived by the owner on May 30, 2022. It is now read-only.

add comments to make springboot can give tips to users #154

Merged
merged 6 commits into from
Feb 25, 2019
Merged

add comments to make springboot can give tips to users #154

merged 6 commits into from
Feb 25, 2019

Conversation

leizhiyuan
Copy link
Contributor

Motivation:

thanks for @chpengzh

image

if write comments as javadoc, spring-boot-configuration-processor will generate descriptions, like this.

Modification:

Describe the idea and modifications you've done.

Result:

Fixes #.

If there is no issue then describe the changes introduced by this PR.

@leizhiyuan leizhiyuan added this to the 6.0.1 milestone Feb 11, 2019
@leizhiyuan leizhiyuan requested review from ujjboy, NeGnail, khotyn and JervyShi and removed request for ujjboy, NeGnail and khotyn February 11, 2019 03:14
Copy link
Member

@JervyShi JervyShi left a comment

Choose a reason for hiding this comment

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

CI failed for missing RpcBindingConverter#loadBalancer

@leizhiyuan
Copy link
Contributor Author

CI failed for missing RpcBindingConverter#loadBalancer

update sofaboot version to fix snapshot

@codecov
Copy link

codecov bot commented Feb 11, 2019

Codecov Report

Merging #154 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##             master    #154   +/-   ##
========================================
  Coverage      57.8%   57.8%           
  Complexity      433     433           
========================================
  Files            56      56           
  Lines          1806    1806           
  Branches        318     318           
========================================
  Hits           1044    1044           
  Misses          554     554           
  Partials        208     208
Impacted Files Coverage Δ Complexity Δ
...ay/sofa/rpc/boot/config/SofaBootRpcProperties.java 56.65% <ø> (ø) 89 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 99033d1...d2626b0. Read the comment docs.

@leizhiyuan leizhiyuan changed the title add commnet to make springboot can give tips to users add comments to make springboot can give tips to users Feb 11, 2019
private Map<String, String> registries = new HashMap<String, String>();

//mesh switch,can be protocol,like bolt,mesh,all,now we only support bolt
/**
* enable mesh,an be protocol,like bolt,mesh,all,now we only support bolt (是否开启mesh支持,目前只支持bolt)
Copy link
Member

Choose a reason for hiding this comment

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

an -> can

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Solved

@leizhiyuan leizhiyuan merged commit 248203f into sofastack:master Feb 25, 2019
@leizhiyuan leizhiyuan deleted the add_comment_to_springboot branch February 25, 2019 03:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants