New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove backward compatibility layer introduced in #6149 #6229
Conversation
do we still want to to use magic negative numbers for |
@spinscale we actually only need to worry about |
LGTM |
@bleskes needs merging, no? |
@bleskes still needs merging? |
hey @bleskes this can be closed as we already removed all of the bw comp code from master right? |
ed0ac86
to
caf0ccf
Compare
caf0ccf
to
bdc10ec
Compare
@javanna not completely. Updated the PR to the latest master. Can you take a look? |
LGTM |
+1 |
PR #6149 changed the
Versions.MATCH_ANY
value from 0 to -3. This was done to allow 0 to be a valid external version. This commit removes the backward compatibility code from v2.0 and up.