[Theme][Enhancement] Review ergonomics of theme selection flags in the push command #3801
Labels
Area: @shopify/theme
@shopify/theme package issues
State: Wontfix
This will not be worked on
tech debt
Liquid SF (tech debt)
-u
,-d
, and--live
represent the role of the theme, so I feel like the combination of these two flags can make usage confusing. Should combining theme push to ALL of those themes?- My initial thought is that it should create 3 themes OR we need to specify the precedence of these flags in the docs
-t
flag changes meaning when you provide it in combination with-u
and-d
theme push -t themename
searches for a theme with the namethemename
and fails if it doesn't existtheme push -u -t themename
will create a theme with the namethemename
even if one already existsThis pr is not the place to apply these changes (in fact, it's kind of difficult to change the semantics of these flags without careful consideration), but I think the ergonomics of these flags can be fleshed out a little more :)
Originally posted by @jamesmengo in #3798 (comment)
The text was updated successfully, but these errors were encountered: