Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement V2 #165

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Implement V2 #165

wants to merge 3 commits into from

Conversation

jeffreyguenther
Copy link
Contributor

@jeffreyguenther jeffreyguenther commented Jan 27, 2025

In this PR, I make the improvements that will ship in V2. This will be the working feature branch. Features will be implement in their own branches and PRs and merged here for a final review.

Shopify has settled on a convention of using `pull` for commands that
get data from their servers. We see this with the new
`theme metafields pull` command.

I think we should be consist. Therefore, we're changing the comand name.
We're removing the theme settings download command, but rather than
break people's setups, we will provide an alias and alert them that they
should change their tooling.
This command is not needed any more. You can achieve a similar result by
parsing the result of the theme list --json command.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant