Optimize configuration management services #5110
Triggered via pull request
September 23, 2024 06:29
hanbingleixue
Status
Cancelled
Total duration
5m 0s
Artifacts
–
maven.yml
on: pull_request
Checkstyle
22s
build
4m 42s
Annotations
8 errors and 1 warning
|
Checkstyle:
sermant-backend/src/main/java/io/sermant/backend/entity/config/ConfigInfo.java#L22
[Checkstyle Check] reported by reviewdog 🐶
'org.apache.commons.lang.StringUtils' should be separated from previous imports.
Raw Output:
/home/runner/work/Sermant/Sermant/./sermant-backend/src/main/java/io/sermant/backend/entity/config/ConfigInfo.java:22:1: error: 'org.apache.commons.lang.StringUtils' should be separated from previous imports. (com.puppycrawl.tools.checkstyle.checks.imports.ImportOrderCheck)
|
|
Checkstyle:
sermant-backend/src/main/java/io/sermant/backend/entity/config/ConfigInfo.java#L137
[Checkstyle Check] reported by reviewdog 🐶
'if' construct must use '{}'s.
Raw Output:
/home/runner/work/Sermant/Sermant/./sermant-backend/src/main/java/io/sermant/backend/entity/config/ConfigInfo.java:137:9: error: 'if' construct must use '{}'s. (com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck)
|
|
Checkstyle:
sermant-backend/src/main/java/io/sermant/backend/entity/config/ConfigInfo.java#L138
[Checkstyle Check] reported by reviewdog 🐶
'if' construct must use '{}'s.
Raw Output:
/home/runner/work/Sermant/Sermant/./sermant-backend/src/main/java/io/sermant/backend/entity/config/ConfigInfo.java:138:9: error: 'if' construct must use '{}'s. (com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck)
|
|
Checkstyle:
sermant-backend/src/main/java/io/sermant/backend/service/ConfigService.java#L140
[Checkstyle Check] reported by reviewdog 🐶
'?' should be on a new line.
Raw Output:
/home/runner/work/Sermant/Sermant/./sermant-backend/src/main/java/io/sermant/backend/service/ConfigService.java:140:65: error: '?' should be on a new line. (com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck)
|
|
Checkstyle:
sermant-backend/src/main/java/io/sermant/backend/service/ConfigService.java#L141
[Checkstyle Check] reported by reviewdog 🐶
':' should be on a new line.
Raw Output:
/home/runner/work/Sermant/Sermant/./sermant-backend/src/main/java/io/sermant/backend/service/ConfigService.java:141:102: error: ':' should be on a new line. (com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck)
|
|
Checkstyle
Process completed with exit code 1.
|
|
build
Canceling since a higher priority waiting request for 'CI-1623-develop' exists
|
|
build
The operation was canceled.
|
|
Checkstyle
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|