Hi,
this PR upgrades to Testcontainers 1.14.1.
Cheers, Christoph
Comment From: snicoll
We are still discussing whether we want to accept certain "only version" upgrades PR, see #20707. I am going to put this one on-hold until we get a chance to finalize this discussion.
Comment From: dreis2211
Sure thing. I kept myself quite for a bit as the version was released two weeks ago already, I think. If that contributes to the discussion a bit.
Comment From: philwebb
We should upgrade this anyway, regardless of #20707. I'm not sure if BOMR will pick it up so I'll drop it in 2.3.0 just so we don't forget.
Comment From: dreis2211
Bomr won't pick it up. That's part of the discussion: if one-liner upgrades should be rejected even though they're not picked up by bomr (which is essentially only stuff in spring-boot-dependencies)
Comment From: snicoll
Thanks for that @dreis2211.
I kept myself quite for a bit as the version was released two weeks ago already, I think.
As I've indicated in this comment, it is less important/urgent to upgrade an internally managed dependency. We do so when we could benefit from a new feature or the dependency fixed an issue we're facing. Otherwise, such upgrade happens in due course as part of a general chore similar to how we apply bomr upgrades.
Comment From: dreis2211
I think the perfect solution is teaching Bomr to handle internal dependencies, too - as suggested in #21439 . That avoids manual work 👍