search

spring-attic / spring-cloud-stream-binder-jms

Other
33 stars 56 forks source link

Pr/26 #29

Open olegz opened 7 years ago

olegz commented 7 years ago

Keep in mind that this commit is continuation of work proposed by 53603d0 commit from @donovanmuller so the two commits must NOT be squashed to preserve proper history

olegz commented 7 years ago

@mbogoevici @viniciusccarvalho @sabbyanandan I believe we're good now. I put DestinationNameResolver and DestinationNames back in even though the modified code is not using it. But as you said, other binders do (at least for now). NOTE: I didn't squash.

mbogoevici commented 7 years ago

@olegz great, thanks! Let's review and merge it next week.