spring-cloud / spring-cloud-function

Apache License 2.0
1.04k stars 618 forks source link

v.4.1.3 source code file contains 4.0.x branch's class file. #1194

Closed Be-poz closed 3 weeks ago

Be-poz commented 1 month ago

I'm not sure if this was intended or not, but the v.4.1.3 source code file contains class files from v.4.0.3.

The class mentioned above is shown below. 4.1.x branch - SmartCompositeMessageConverter 4.0.x branch - SmartCompositeMessageConverter

I'm guessing this is because the class in the 4.1.x branch was committed a month ago, and the 4.1.3. release was about 4 months ago, is that correct? If that's correct, I'd like to make sure that the 4.1.4 release contains this commit.

olegz commented 1 month ago

I am not sure what are you asking. Can you please clarify? SmartCompositeMessageConverter is one of the core components of the framework, so it's in all branches

Be-poz commented 1 month ago
image

I wanted to say that the source code for the SmartCompositeConverter in the 4.1.3 version is not from the 4.1.x branch, but from 4.0.x. I also wanted to make sure that this was because a commit was made to the SmartCompositeConverter code in the 4.1.x branch after the 4.1.3 release, and I wondered if the 4.1.4 release would reflect that commit.

olegz commented 1 month ago

Yes, but i still don't understand what you are asking and what commit you are referring to. You posted an image of the directory structure which doesn't show anything.

Be-poz commented 1 month ago

What I was confused about at first

What I guessed

What I wanted to ask

It's pretty obvious that this will be in the next release, but I wanted to ask just to be sure. Thanks. You can close this issue.

olegz commented 3 weeks ago

I am still confused about what you are asking and what source.zip you are referring to. The fix has been added to 4.1.x - https://github.com/spring-cloud/spring-cloud-function/commit/6dee668e3ec20710460b0384a8b6afe53d078633 as well as 4.2.x https://github.com/spring-cloud/spring-cloud-function/commit/571783ffae5f24df9fbc0cc08a2e3838c97b58ae So yes they will be included in the next release. 4.1.x will be released on November 12