INT-406: fix move CLI version checks - #1593
Conversation
|
@AlexanderLiteplo and I had a discussion about this. So the entire stack has been tested at As such we think instead of just allowing anyone over the version of |
|
Got it. I'm assuming we can close this PR then and we can open new PR for adding warning @shankars99 ? |
|
Good catch Dan, I think we should merge this as that is indeed a bug. |
|
We can add a warning in a new PR or just add a warning to this one. |
|
I recently tested with version over 7 and everything worked |
AlexanderLiteplo
left a comment
There was a problem hiding this comment.
THis should be merged, and then we can put a warning within the CLI later. For now the Readme reccomends 6.0.1 which is enough of a warning. But currently there is a blocked partner so lets get this merged asap.
|
mvm going to just quickly add a warning when its not 6.0.1 |
…ended versions in config utility. Add corresponding test case for warning message format.
Added warning cc: @shankars99 @DanL0 |
Co-authored-by: alexanderliteplo <alexanderliteplo@gmail.com>
Co-authored-by: alexanderliteplo <alexanderliteplo@gmail.com>


Based on: #1592
The version check for the CLI was wrong, as flagged by the original PR creator it was showing that 7.5.0 isn't greater or equal than 6.0.2.
This PR fixes the problem and also adds tests and enables running them in CI.