Skip to content
This repository was archived by the owner on Aug 3, 2023. It is now read-only.

Change '-V' to '-v' in wrangler output in template - #2341

Merged
JacobMGEvans merged 1 commit into
cloudflare:masterfrom
Divkix:patch-1
Dec 29, 2022
Merged

Change '-V' to '-v' in wrangler output in template#2341
JacobMGEvans merged 1 commit into
cloudflare:masterfrom
Divkix:patch-1

Conversation

@Divkix

@Divkix Divkix commented Dec 24, 2022

Copy link
Copy Markdown
Contributor

No description provided.

@Divkix
Divkix requested a review from a team as a code owner December 24, 2022 19:29
@JacobMGEvans
JacobMGEvans merged commit d7c9d3f into cloudflare:master Dec 29, 2022
@Divkix
Divkix deleted the patch-1 branch December 29, 2022 18:25
@mrbbot

mrbbot commented Jan 3, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

Hey! This repo is for Wrangler 1, which does use -V for getting the version. Using -v will result in an error. We should revert this PR, or maybe ask users to try both?

$ wrangler -v     
error: Found argument '-v' which wasn't expected, or isn't valid in this context

USAGE:
    wrangler [FLAGS] [OPTIONS] <SUBCOMMAND>

For more information try --help

$ wrangler -V
wrangler 1.20.0
mrbbot added a commit that referenced this pull request Jan 3, 2023
@mrbbot
Revert "Change '-V' to '-v' in wrangler output in templatw (#2341)"
c022854 
This reverts commit d7c9d3f.
mrbbot added a commit that referenced this pull request Jan 3, 2023
@mrbbot
Revert #2341, and suggest Wrangler 2 if wrangler -V fails
def9156
@mrbbot mrbbot mentioned this pull request Jan 3, 2023
Merged
mrbbot added a commit that referenced this pull request Jan 3, 2023
@mrbbot
Revert #2341, and suggest Wrangler 2 if wrangler -V fails (#2342)
d86d1c7
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

3 participants

@Divkix @mrbbot @JacobMGEvans