Ah, I figured it could make sense to let the message logic be out of scope of the more narrow purpose of the function, tightly scoped to just get the version as compared to also including log messages and possible logic on what to do if the version is too low etc.
I'm not confident on a change to make as a followup myself, but I'm happy to merge a change you think makes sense!
Ah, I figured it could make sense to let the message logic be out of scope of the more narrow purpose of the function, tightly scoped to just get the version as compared to also including log messages and possible logic on what to do if the version is too low etc.
I'm not confident on a change to make as a followup myself, but I'm happy to merge a change you think makes sense!