-
Notifications
You must be signed in to change notification settings - Fork 8.5k
✨ support spring-boot 4.0.0、spring-cloud 2025.1.0 #4068
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
Conversation
9ed9cf4 to
9a1da2c
Compare
|
@uuuyuqi 请问一下,如何解决这个代码格式的问题?
|
|
你好,2013-present 是更加合理的。我会尽快提交一个 PR 先对这些内容统一进行修改,方便后续大家提PR。 |
|
@uuuyuqi 好的,谢谢,到时候我再重新处理一下这个 PR |
|
我仔细看了下这个 style check 报错,目前看下来是 SpringCloud 社区从 2025.0.0 演进到 2025.1.0 时,修改了 spring-cloud-build-tools/src/main/resources/checkstyle-header.txt 校验内容,从 不过目前 SpringCloud 2025.1.x 尚未正式发布,SpringCloudAlibaba 也会优先发布 2025.0.x 版本,大概率就10月份,具体时间暂时无法确定。cc @LearningGp 这个 PR 的审查和合并会先搁置,等到 2025.0.0 发布后会继续启动,到时候我们一起讨论下这里的 style 如何修改更合适~ |
|
@uuuyuqi OK |
752d367 to
610d164
Compare
3a30788 to
a6614fb
Compare
|
@uuuyuqi 前两天已经重新整理此 pr |
f6bf8b0 to
5b2c62b
Compare
5b2c62b to
905168d
Compare
1. disable RocketMQMessageChannelBinderTest 2. enable FeignClientCircuitBreakerRuleIntegrationTest
|
@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
b0d40d7 to
6324deb
Compare
|
@xuxiaowei-com-cn 2025.1.0 预计什么时候可以发布 |
|
@kuangyoubo 我只是项目一个普通贡献者,并没有此项目的管理权限。 |
|
社区还在收集相关的问题,修复这些问题后会先发布2025.0.0.1,然后再处理 2025.1.0.0 版本,这两个版本目前都预计在12月份完成 |
spring boot 4升级了很多东西,想尝试下spring boot的RequestMapping version,期待2025.0.0.1早日发布 |
谢谢,先用你的方式适配到。我在做老项目迁移,结果在这里卡住了,哈哈哈 |
|
please add support for spring-cloud-gateway-server-mvc which benefit from virtual threads. @xuxiaowei-com-cn ❤️
|
|
|
@uuuyuqi 我想问一下,代码审核就这么随意吗?看一下 #4146 这个 PR,还有我的评论:#4146 (comment) ,看一下当前的这个 PR, |
|
LGTM. Really appreciate your contribution! |
|

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