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

Simplify hlx and move operational commands to hops #927

Closed
tripodsan opened this issue May 27, 2019 · 5 comments
Closed

Simplify hlx and move operational commands to hops #927

tripodsan opened this issue May 27, 2019 · 5 comments
Labels
enhancement New feature or request

Comments

@tripodsan
Copy link
Contributor

tripodsan commented May 27, 2019

Overview

for a new user, the current hlx command line is very convoluted. for simple local development, the available commands should be hidden or moved to helix-ops aka hops.

local dev commands

$ hlx --help
Usage: hlx <command> [options]

Commands:
  hlx demo <name> [dir]  Create example helix project.
  hlx up [files...]      Run a Helix development server
  hlx clean              Remove generated files and caches.
  hlx completion         generate completion script

operations commands

$ hops --help
Usage: hops <command> [options]

Commands:
  hlx build [files..]    Compile the template functions and build package
  hlx package            Create Adobe I/O runtime packages
  hlx deploy             Deploy packaged functions to Adobe I/O runtime
  hlx perf               Test performance
  hlx publish            Activate strains in the Fastly CDN and publish the site
  hlx clean              Remove generated files and caches.
  hlx auth               Authenticate against 3rd party systems for development
                         and deployment
  hlx completion         generate completion script

1. stage

  • create new binary entry-point hops
  • hops will support all the commands that hlx supports today
  • hlx will hide the advanced commands, but still support them

2. stage

  • hops will only support the operations commands (eg. not up)
  • hlx will only support the non operations command

Alternative names

  • hops
  • hlxops
@tripodsan tripodsan added the enhancement New feature or request label May 27, 2019
@filmaj
Copy link
Contributor

filmaj commented May 27, 2019

Splitting this into two commands seems convoluted to me 🤷‍♂

@trieloff
Copy link
Contributor

hops should also have a completion script

@trieloff
Copy link
Contributor

@filmaj the idea is that you won't see hops unless you check your CI script or perform troubleshooting. We want to hide almost everything that's in hops behind automation.

@bdelacretaz
Copy link

+1 for these changes along with #241 and mentioning adobe/helix-home#40 here, the developer experience theme.

@tripodsan
Copy link
Contributor Author

won't fix for helix 2. irrelevant for helix 3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants