Repository navigation
fix(cli): reject dev --remote command - #278
Conversation
|
I'm Bonk, and I've done a quick review of your PR. Adds early rejection for
|
dom96
left a comment
There was a problem hiding this comment.
This seems fine, though I think it would make most sense to add this error message in wrangler (just in case someone runs wrangler instead of pywrangler)
Well wrangler still supports this feature. It currently warns users to user remote bindings instead, but probably they don't want to suddenly stop supprting this feature. |
|
I mean add the error message just for Python Workers in Wrangler. |
|
Ah, I see. Yeah, probably. It is a little bit unclear which part should live in wrangler and which part should live in pywrangler though. |
We had a few cases where users were using
pywrangler dev --remoteand getting errors. This adds a clear error message that it is not supported and forwarding them to use remote bindings instead.