Skip to content

Conversation

@haydenrear
Copy link

Closes gh-36043

Adds ability to change message converters list before it's returned by registering consumer using configureMessageConvertersList

Signed-off-by: Hayden Rear [email protected]

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Dec 19, 2025
@haydenrear haydenrear force-pushed the feature/add-message-converter-list-configurer branch 3 times, most recently from c67f3aa to aeae7b9 Compare December 19, 2025 01:46
@bclozel
Copy link
Member

bclozel commented Dec 29, 2025

@haydenrear thanks for this PR, but we cannot process it at the moment.
Can you:

  • squash and sign the commits into one?
  • run the build locally and ensure it passes?

Thanks

@bclozel bclozel added the status: waiting-for-feedback We need additional information before we can continue label Dec 29, 2025
Adds ability to add message converter at specific index to builders, and propagates through to java.util.List.add(idx, ...)

Signed-off-by: Hayden Rear [email protected]
@haydenrear haydenrear force-pushed the feature/add-message-converter-list-configurer branch from aeae7b9 to c9eafda Compare December 30, 2025 01:27
@haydenrear
Copy link
Author

@haydenrear thanks for this PR, but we cannot process it at the moment. Can you:

  • squash and sign the commits into one?
  • run the build locally and ensure it passes?

Thanks

Hi @bclozel

I have squashed and ran the build locally and pushed.

Thanks!
Hayden

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels Dec 30, 2025
@bclozel
Copy link
Member

bclozel commented Dec 30, 2025

Thanks @haydenrear but it seems that your commits aren't signed with the DCO (even though it looks like they are in the commit message). At this stage, I'm not sure how we will resolve #36043 so I'll close this PR to avoid spending time processing a PR that we cannot merge.

@bclozel bclozel closed this Dec 30, 2025
@bclozel bclozel added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged or decided on status: feedback-provided Feedback has been provided labels Dec 30, 2025
@haydenrear haydenrear deleted the feature/add-message-converter-list-configurer branch December 30, 2025 15:44
@haydenrear haydenrear restored the feature/add-message-converter-list-configurer branch December 30, 2025 16:01
@haydenrear haydenrear deleted the feature/add-message-converter-list-configurer branch December 30, 2025 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: declined A suggestion or change that we don't feel we should currently apply

Projects

None yet

3 participants