fix(upgrade) Fix basicSemverOperatorRegex

[shilcare]

Summary

Fix bug #7079 of basicSemverOperatorRegex. It doesn't match the range operator correctly if the dependency range is specified with >=. For example, when running yarn upgrade --latest, if a dependency is specified with >=, the matched result will be >, which is not correct.

This is because the regular expression engine looks for alternations one-by-one. We can fix it by putting >= before >.

fixes #7079

Test plan

[rally25rs]

Nice catch! Thanks for the contribution. 🎉
Could you please add a line to CHANGELOG.md for this change?

[rally25rs]

You know what, maybe we should just take > out of the regex. I think it actually causes another bug. If yarn upgrades you to a "latest" of 1.0.0 and preserves the > then it will change your pacakge.json to >1.0.0 which makes the latest version not acceptable. 🤦‍♂️ oof.

[shilcare]

You know what, maybe we should just take > out of the regex. I think it actually causes another bug. If yarn upgrades you to a "latest" of 1.0.0 and preserves the > then it will change your pacakge.json to >1.0.0 which makes the latest version not acceptable. 🤦‍♂️ oof.

I think you're right :). We can't handle > with current buildPatternToUpgradeTo implementation.

[rally25rs]

The continuous-integration/appveyor/pr status isn't reporting correctly, but the build finished and passed: https://ci.appveyor.com/project/kittens/yarn/builds/22821129

Going to merge this PR.