-
Notifications
You must be signed in to change notification settings - Fork 60
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
release 3.0.0-rc2 #568
release 3.0.0-rc2 #568
Conversation
update java-sdk version
…CO-BCOS#550) * <fix>(console): fix bfs command completer. * <fix>(jline): fix bfs commands completer.
update java sdk version
update log4j2 version to 2.17.1
<fix>(console): fix auth init bug, fix cd command bug.
<fix>(console): fix wasm getCode bug, fix get transaction receipt bug.
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Sider has detected 1 warning on analyzing the commit 1eb3e18.
If the errors persist even after retrying, the following actions may resolve them:
If you still have problems, feel free to ask us via chat. 💬 You can turn off such notifications if unnecessary. |
if (param.startsWith("~")) { | ||
pathToFix = "/apps/" + param.substring(1); | ||
} else { | ||
pathToFix = pwd + ((pwd.equals("/")) ? "" : "/") + param; |
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.
[Checkstyle] String literal expressions should be on the left side of an equals comparison. (view)
Rule | Severity |
---|---|
EqualsAvoidNullCheck |
info |
References:
You can close this issue if no need to fix it. Learn more.
Tuple2<Integer, Integer> childCount = | ||
travelBfs( | ||
absolutePath | ||
+ (absolutePath.equals("/") ? "" : "/") |
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.
[Checkstyle] String literal expressions should be on the left side of an equals comparison. (view)
Rule | Severity |
---|---|
EqualsAvoidNullCheck |
info |
References:
You can close this issue if no need to fix it. Learn more.
(consoleInitializer, params, pwd) -> | ||
consoleInitializer.getPrecompiledFace().remove(params[0])); | ||
*/ | ||
// TODO: Table CRUD service is not supported in FISCO BCOS 3.0.0 rc1 |
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.
[Checkstyle] Comment matches to-do format 'TODO:'. (view)
Rule | Severity |
---|---|
TodoCommentCheck |
info |
References:
You can close this issue if no need to fix it. Learn more.
@@ -58,8 +58,10 @@ public static void main(String[] args) { | |||
} | |||
|
|||
WelcomeInfo.welcome(); | |||
String pwd = "/"; | |||
SupportedCommand.setIsAuthOpen(consoleInitializer.getClient().isAuthCheck()); | |||
String pwd = "/apps"; |
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.
[PMD Java] The method 'main(String)' has a cognitive complexity of 81, current threshold is 15 (view)
Rule | Ruleset | Priority |
---|---|---|
CognitiveComplexity |
Design | 3 |
References:
You can close this issue if no need to fix it. Learn more.
// CnsInfo latestCNSInfo = cnsInfos.get(cnsInfos.size() - 1); | ||
// contractAddress = latestCNSInfo.getAddress(); | ||
// contractAbi = latestCNSInfo.getAbi(); | ||
List<CnsInfo> cnsInfos = cnsService.selectByName(contractName); |
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.
[PMD Java] The method 'callByCNS(String)' has a cognitive complexity of 17, current threshold is 15 (view)
Rule | Ruleset | Priority |
---|---|---|
CognitiveComplexity |
Design | 3 |
References:
You can close this issue if no need to fix it. Learn more.
@Override | ||
public void pwd(String pwd) { | ||
System.out.println(pwd); | ||
} | ||
|
||
private Tuple2<Integer, Integer> travelBfs( |
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.
[PMD Java] The method 'travelBfs(String, String, int, int)' has a cognitive complexity of 17, current threshold is 15 (view)
Rule | Ruleset | Priority |
---|---|---|
CognitiveComplexity |
Design | 3 |
References:
You can close this issue if no need to fix it. Learn more.
(consoleInitializer, params, pwd) -> | ||
consoleInitializer.getPrecompiledFace().remove(params[0])); | ||
*/ | ||
// TODO: Table CRUD service is not supported in FISCO BCOS 3.0.0 rc1 |
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.
[PMD Java] Avoid really long classes. (view)
Rule | Ruleset | Priority |
---|---|---|
ExcessiveClassLength |
Design | 3 |
References:
You can close this issue if no need to fix it. Learn more.
@@ -283,7 +283,7 @@ public static void deployHelp(boolean isWasm) { | |||
System.out.println( | |||
"* abi -- The path of ABI file after contract being compiled via cargo-liquid."); | |||
System.out.println( | |||
"* path -- The path of BFS where the contract will be located at, such as /apps/liquid/YouContract/"); | |||
"* path -- The path of BFS where the contract will be located at, such as '/apps/liquid/YouContract/'."); |
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.
[PMD Java] This class has a bunch of public methods and attributes (view)
Rule | Ruleset | Priority |
---|---|---|
ExcessivePublicCount |
Design | 3 |
References:
You can close this issue if no need to fix it. Learn more.
@@ -971,13 +984,13 @@ public void printUsageInfo() { | |||
LIST_DIR.getCommand(), | |||
CHANGE_DIR.getCommand(), | |||
MAKE_DIR.getCommand(), | |||
PWD.getCommand())); | |||
TREE.getCommand())); |
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.
[PMD Java] Do not use non-final non-private static fields (view)
Rule | Ruleset | Priority |
---|---|---|
MutableStaticState |
Design | 3 |
References:
You can close this issue if no need to fix it. Learn more.
|
||
// TODO: Table CRUD service is not supported in FISCO BCOS 3.0.0 rc1 | ||
public static List<String> CRUD_COMMANDS = | ||
new ArrayList<>( | ||
Arrays.asList( | ||
// CREATE.getCommand(), | ||
CREATE.getCommand() |
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.
[PMD Java] Do not use non-final non-private static fields (view)
Rule | Ruleset | Priority |
---|---|---|
MutableStaticState |
Design | 3 |
References:
You can close this issue if no need to fix it. Learn more.
No description provided.