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

cli: Refactor build and install commands #22538

Merged
merged 1 commit into from
Jan 27, 2022

Commits on Jan 27, 2022

  1. cli: Refactor build and install commands

    This lays some groundwork for future PRs that will run builds in
    parallel, prefixing each line output with the appropriate slug.
    
    The helpers in `helpers/tasks/installProjectTask.js` weren't all that
    helpful anymore. Instead we provide some lower-level functions in
    `helpers/install.js` that can be built upon for the slightly different
    future requirements of install versus build.
    
    Install no longer confusingly reports that it's installing pnpm
    dependencies for every package being installed, instead doing a single
    `pnpm install` task at the top level.
    
    Install no longer behaves weirdly when passed both `--root` and `--all`.
    
    Build gets an `--all` option (because it was easy to add), and an option
    to skip the `pnpm install`.
    
    Build also handles its own output in verbose mode (rather than using
    listr-verbose-renderer) in preparation for parallel builds.
    
    Due to the abstracting of `jetpack install --all` and `jetpack build
    packages`, those commands now also accept multiple packages on the
    command line.
    anomiex committed Jan 27, 2022
    Configuration menu
    Copy the full SHA
    a7ba840 View commit details
    Browse the repository at this point in the history