fix: isRestDeploy correctly handles org-metadata-rest-deploy config c…

[WillieRuemmele]

…onversion

What does this PR do?

isRestDeploy method respects the sf/sfdx config value conversion

comments in the method are now correct. SOAP is default, unless the config/env var is true

What issues does this PR fix or reference?

@W-11390183@

[mshanemc]

QA notes

✅ ./../plugin-source/bin/dev force:source:deploy -p force-app deploys soap
sfdx config:set restDeploy=true and repeat of the same command
✅ *** Deploying with REST API v55.0 ***
[due to infra issues, I'm not waiting for these deploys to finish. I can confirm the api choice from the ui]

source:convert and ../../plugin-source/bin/dev force:mdapi:deploy --deploydir mdapiOut --wait 100
✅ *** Deploying with REST *** (config is still set)
../../plugin-source/bin/dev force:mdapi:deploy --deploydir mdapiOut --wait 100 --soapdeploy
✅ *** Deploying with SOAP ***
sfdx config:unset restDeploy
../../plugin-source/bin/dev force:mdapi:deploy --deploydir mdapiOut --wait 100
✅ *** Deploying with SOAP ***

../../plugin-source/bin/dev force:source:delete -p force-app/main/default/classes/GeocodingServiceTest.cls
✅ *** Deleting with SOAP API ***

sfdx config:set restDeploy=true
../../plugin-source/bin/dev force:source:delete -p force-app/main/default/classes/TestPropertyController.cls

✅ *** Deleting with REST API ***