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.

Sign up for GitHub

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

Jump to bottom

[Themes] - Refactor pull command to prepare for public themes API #4500

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

[Themes] - Refactor pull command to prepare for public themes API #4500

wants to merge 4 commits into from
+236 −48

Commits on Sep 23, 2024

  1. Move theme pull logic from command layer to service layer 

    Update tests

Move + fix CLI2 tests

Remove pull command test

Remove CLI2 Invocation + Tests

Remove legacy flag

Add JSDoc to pull export

Export public pull service from themes package

Revert change to theme-command
    @jamesmengo
    jamesmengo committed Sep 23, 2024
    Configuration menu
    Copy the full SHA
    ef6ba7d View commit details
    Browse the repository at this point in the history

  2. Add noColor and verbose flags

    @jamesmengo
    jamesmengo committed Sep 23, 2024
    Configuration menu
    Copy the full SHA
    2517473 View commit details
    Browse the repository at this point in the history

  3. Update JSDoc

    @jamesmengo
    jamesmengo committed Sep 23, 2024
    Configuration menu
    Copy the full SHA
    4502916 View commit details
    Browse the repository at this point in the history

  4. Add CLI2 invocation back to pull command 

    Re-arrange + export ThemeCommand helpers
    @jamesmengo
    jamesmengo committed Sep 23, 2024
    Configuration menu
    Copy the full SHA
    c2bceec View commit details
    Browse the repository at this point in the history