Skip to content

Conversation

@xuxiaowei-com-cn
Copy link
Contributor

Describe what this PR does / why we need it

#4067

Does this pull request fix one issue?

Describe how you did it

Describe how to verify it

Special notes for reviews

@xuxiaowei-com-cn
Copy link
Contributor Author

@uuuyuqi 请问一下,如何解决这个代码格式的问题?

  1. 如果我将 Copyright 2013-2023 the original author or authors. 替换为 Copyright 2013-present the original author or authors. xuxiaowei-com-cn@752d367 (修改了 452 个文件) 确实可以解决此问题,但是修改的文件数量太多了。
  2. 如果不修改 Copyright 2013-2023 the original author or authors. 则会出现问题:https://github.com/alibaba/spring-cloud-alibaba/actions/runs/18028214112
  3. 如果有其他方案,我会重新整理提交的代码。
image

@uuuyuqi
Copy link
Collaborator

uuuyuqi commented Sep 28, 2025

你好,2013-present 是更加合理的。我会尽快提交一个 PR 先对这些内容统一进行修改,方便后续大家提PR。

@xuxiaowei-com-cn
Copy link
Contributor Author

@uuuyuqi 好的,谢谢,到时候我再重新处理一下这个 PR

@uuuyuqi
Copy link
Collaborator

uuuyuqi commented Sep 28, 2025

我仔细看了下这个 style check 报错,目前看下来是 SpringCloud 社区从 2025.0.0 演进到 2025.1.0 时,修改了 spring-cloud-build-tools/src/main/resources/checkstyle-header.txt 校验内容,从 Copyright \E20\d\d\-20\d\d\Q 改为了 Copyright \E20\d\d\-present\Q。SpringCloudAlibaba 升级到 SpringCloud 2025.1.x 时也会升级 spring-cloud-build-tools 版本,也就会出现这个 style check 报错,感谢 @xuxiaowei-com-cn 的发现!后续 SpringCloudAlibaba 社区会 follow 这个变更。

不过目前 SpringCloud 2025.1.x 尚未正式发布,SpringCloudAlibaba 也会优先发布 2025.0.x 版本,大概率就10月份,具体时间暂时无法确定。cc @LearningGp

这个 PR 的审查和合并会先搁置,等到 2025.0.0 发布后会继续启动,到时候我们一起讨论下这里的 style 如何修改更合适~

@xuxiaowei-com-cn
Copy link
Contributor Author

@uuuyuqi OK

@xuxiaowei-com-cn
Copy link
Contributor Author

@uuuyuqi 前两天已经重新整理此 pr

@xuxiaowei-com-cn xuxiaowei-com-cn changed the base branch from 2025.0.x to 2025.1.x October 29, 2025 06:08
@xuxiaowei-com-cn xuxiaowei-com-cn force-pushed the 2025.x branch 2 times, most recently from f6bf8b0 to 5b2c62b Compare October 30, 2025 08:34
@xuxiaowei-com-cn xuxiaowei-com-cn changed the title ✨ support spring-cloud 2025.1.x ✨ support spring-boot 4.0.0、spring-cloud 2025.1.x Nov 6, 2025
@xuxiaowei-com-cn xuxiaowei-com-cn changed the title ✨ support spring-boot 4.0.0、spring-cloud 2025.1.x ✨ support spring-boot 4.0.0、spring-cloud 2025.1.0 Nov 16, 2025
1. disable RocketMQMessageChannelBinderTest
2. enable FeignClientCircuitBreakerRuleIntegrationTest
@xuxiaowei-com-cn
Copy link
Contributor Author

xuxiaowei-com-cn commented Nov 24, 2025

@uuuyuqi @LearningGp 您好,spring-boot 4.0.0、spring-cloud 2025.1.0 已经发布正式版,麻烦您有时间处理一下。

1. internal classes use static final modifier
2. fix nacos instance url
3. fix NacosAutoServiceRegistration Bean
4. fix NacosDiscoveryLoadBalancerConfigurationTest

alibaba#4067
@kuangyoubo
Copy link

@xuxiaowei-com-cn 2025.1.0 预计什么时候可以发布

@xuxiaowei-com-cn
Copy link
Contributor Author

@kuangyoubo 我只是项目一个普通贡献者,并没有此项目的管理权限。
如果你着急用,可以参考 #4067 (comment) ,使用我自己编译的版本。

@LearningGp
Copy link
Collaborator

社区还在收集相关的问题,修复这些问题后会先发布2025.0.0.1,然后再处理 2025.1.0.0 版本,这两个版本目前都预计在12月份完成

@shaobin0613
Copy link

社区还在收集相关的问题,修复这些问题后会先发布2025.0.0.1,然后再处理 2025.1.0.0 版本,这两个版本目前都预计在12月份完成

spring boot 4升级了很多东西,想尝试下spring boot的RequestMapping version,期待2025.0.0.1早日发布

@kuangyoubo
Copy link

@kuangyoubo 我只是项目一个普通贡献者,并没有此项目的管理权限。 如果你着急用,可以参考 #4067 (comment) ,使用我自己编译的版本。

谢谢,先用你的方式适配到。我在做老项目迁移,结果在这里卡住了,哈哈哈

@lcxw
Copy link

lcxw commented Dec 15, 2025

please add support for spring-cloud-gateway-server-mvc which benefit from virtual threads. @xuxiaowei-com-cn ❤️

  1. spring-cloud-gateway-server-mvc for spring-cloud-alibaba with nacos config support

@lcxw
Copy link

lcxw commented Dec 15, 2025

#3916

@xuxiaowei-com-cn
Copy link
Contributor Author

@lcxw

  1. spring-cloud-starter-gateway-server-webflux 是 webflux
  2. spring-cloud-starter-gateway-server-webmvc 是 mvc
  3. spring-cloud-alibaba 之前只支持 gateway webflux
  4. 如果要添加 gateway mvc,需要增加一个 Maven 模块(坐标)
  5. spring cloud alibaba支持spring cloud 2023 gateway mvc 模式吗 #3916 议题很久没有处理,并且据我观察 gateway mvc 并不受重视
  6. 而且 gateway 不建议使用 mvc,不如 webflux 性能高
  7. 如果要增加一个 Maven 模块(坐标),需要和项目管理者商议一下,我只是一个普通贡献者,这个做不了主

@xuxiaowei-com-cn
Copy link
Contributor Author

@uuuyuqi 我想问一下,代码审核就这么随意吗?看一下 #4146 这个 PR,还有我的评论:#4146 (comment) ,看一下当前的这个 PR,

@fangjian0423
@flystar32
@mercyblitz
@DanielLiu1123

@uuuyuqi
Copy link
Collaborator

uuuyuqi commented Dec 16, 2025

LGTM. Really appreciate your contribution!

@uuuyuqi uuuyuqi merged commit 89436ae into alibaba:2025.1.x Dec 16, 2025
4 checks passed
@xuxiaowei-com-cn
Copy link
Contributor Author

  1. alibaba/spring-cloud-alibaba 官方 GitHub Package Maven 快照已发布,建议大家使用官方快照。
  2. 没有特别情况下,个人快照 https://gitlab.xuxiaowei.com.cn/xuxiaowei-com-cn/spring-cloud-alibaba/ 不再更新。

#4185

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants