gh-cli-for-education / gh-edu

A pluggable extension of gh-cli to assist educators and students using GitHub Education
https://github.com/gh-cli-for-education/TFG-2122-Cristo-Garcia-Gonzalez
3 stars 0 forks source link

gh edu install (no args) #3

Open crguezl opened 2 years ago

crguezl commented 2 years ago

What do you think about this @GGCristo ?

We will keep of the plugins installed with gh edu install plugin inside the config.json file and then ...

When called without arguments:

gh edu install

should install (or update?) the plugins specified in file config.json.

GGCristo commented 2 years ago

We have a gh edu update command (Maybe it should be called upgrade instead). We can reuse this command to also update plugins, not only the cache. Also, gh cli doesn't have this behavior so it could be unintuitive.

crguezl commented 2 years ago

@GGCristo

We have a gh edu update command (Maybe it should be called upgrade instead). We can reuse this command to also update plugins, not only the cache. Also, gh cli doesn't have this behavior so it could be unintuitive.

Agree. May be gh edu update can be multi-command like:

gh edu update cache    # updates the cache
gh edu update plugins  # updates the plugins
etc.
crguezl commented 2 years ago

Or may be just an option of gh edu update:

program
  .command("update")
  .option("-c, --cache", "Update your cache")
  .option("-p, --plugins", "Update your plugins")
  .option("-a, --all", "Update all")
  .description("Update your configuration")
  .action((options) => {
    update(options);
  })

In any case an user can always update manually, so may be this requirement is not so important and can be delayed