optimize: update checkStyle version#13914
Conversation
|
Thanks for your this PR. 🙏 感谢您提交的PR。 🙏 |
|
@WangzJi 有个文件有冲突, 麻烦解决一下 |
# Conflicts: # ai/src/test/java/com/alibaba/nacos/ai/service/McpServerTransformServiceTest.java
|
@WangzJi 好像修改了之后, 有checkstyle无法通过的问题 |
|
The code in this PR does not modify this part, so I’m not sure why the PMD failure is showing up. |
|
应该是#13878 这个pr引入的,之前合并的时候不知道为什么没跑出来。 帮忙先加个忽略标签跳过一下。 这两个报警的方法上都加一下 |
|
McpServerTransformServiceTest.java 似乎有很多测试不通过的, 我用develop分支跑是可以通过的 |
|
|
|
|
Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
closes: #13796
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:
[ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=trueto make sure basic checks pass. Runmvn clean install -DskipITsto make sure unit-test pass. Runmvn clean test-compile failsafe:integration-testto make sure integration-test pass.