Add pre-release (#29)

Author: smashedr

README.md

@@ -97,7 +97,13 @@ Install the latest release.
 ir owner/repo
 ```
 
-Install a specific version.
+Include pre-releases.
+
+```shell
+ir owner/repo --pre
+```
+
+Install a specific version/tag.
 
 ```shell
 ir owner/repo v4.2.0
@@ -135,10 +141,22 @@ ir list
 
 Remove an app.
 
+```shell
+ir remove
+```
+
+Remove an app by name.
+
 ```shell
 ir remove name
 ```
 
+Edit the settings.
+
+```shell
+ir config
+```
+
 [![View Documentation](https://img.shields.io/badge/view_documentation-blue?style=for-the-badge&logo=googledocs&logoColor=white)](https://smashedr.github.io/install-release/)
 
 # Development

cmd/info.go

@@ -13,24 +13,28 @@ import (
 )
 
 var infoCmd = &cobra.Command{
-	Use:     "info [owner/repo]",
+	Use:     "info [owner/repo] [tag]",
 	Aliases: []string{"i", "in", "inf"},
 	Short:   "Show app or package information",
 	Long:    "Show app or package information.",
 	Run: func(cmd *cobra.Command, args []string) {
 		binPath := viper.GetString("bin")
-		//sumFlag, _ := cmd.Flags().GetBool("summary")
-		log.Debug("infoCmd", "args", args, "binPath", binPath)
+		preRelease, _ := cmd.Flags().GetBool("pre")
+		log.Debug("infoCmd", "args", args, "binPath", binPath, "preRelease", preRelease)
 
 		if len(args) >= 1 && strings.Contains(args[0], "/") {
 			owner, repo, err := parseRepository(args[0])
 			if err != nil {
 				_ = cmd.Help()
 				log.Fatal(err)
 			}
+			tag := "latest"
+			if len(args) > 1 {
+				tag = args[1]
+			}
 			log.Info("Repository", "owner", owner, "repo", repo)
 			client := getClient()
-			release, err := getRelease(client, owner, repo, "")
+			release, err := getRelease(client, owner, repo, tag, preRelease)
 			if err != nil {
 				log.Fatalf("Error getting release: %v", err)
 			}

cmd/install.go

@@ -32,13 +32,13 @@ var archAliases = map[string][]string{
 
 func runInstall(cmd *cobra.Command, args []string) error { // NOSONAR
 	cmd.SilenceUsage = true // set here so subcommands do not silence usage
-	var err error
 	binPath := viper.GetString("bin")
 	log.Debug("runInstall", "args", args, "binPath", binPath)
-	skipPrompts, _ := cmd.Flags().GetBool("yes")
-	assetName, _ := cmd.Flags().GetString("asset")
-	destName, _ := cmd.Flags().GetString("name")
-	log.Info("Flags:", "skipPrompts", skipPrompts, "assetName", assetName, "destName", destName)
+	skipPrompts := viper.GetBool("yes")
+	assetName := viper.GetString("asset")
+	destName := viper.GetString("name")
+	preRelease, _ := cmd.Flags().GetBool("pre")
+	log.Info("Flags:", "skipPrompts", skipPrompts, "assetName", assetName, "destName", destName, "preRelease", preRelease)
 
 	if len(args) < 1 {
 		_ = cmd.Help()
@@ -48,7 +48,7 @@ func runInstall(cmd *cobra.Command, args []string) error { // NOSONAR
 	owner, repo, err := parseRepository(args[0])
 	if err != nil {
 		_ = cmd.Help()
-		log.Fatal(err)
+		return err
 	}
 
 	tag := "latest"
@@ -61,15 +61,15 @@ func runInstall(cmd *cobra.Command, args []string) error { // NOSONAR
 
 	client := getClient()
 
-	release, err := getRelease(client, owner, repo, tag)
+	release, err := getRelease(client, owner, repo, tag, preRelease)
 	if err != nil {
 		return fmt.Errorf("get release error: %w", err)
 	}
 	if verbose >= 3 {
 		log.Debugf("release: %v", release)
 	}
 
-	styles.PrintKV("Version:", release.GetTagName())
+	styles.PrintKV("Version:", fmt.Sprintf("%s (%s)", release.GetTagName(), release.GetName()))
 
 	// Asset
 	var asset *github.ReleaseAsset
@@ -198,7 +198,7 @@ func runInstall(cmd *cobra.Command, args []string) error { // NOSONAR
 			}).
 			Value(&destName)
 		if err := form.Run(); err != nil {
-			log.Fatalf("Prompt failed %v", err)
+			return err
 		}
 		log.Debugf("4 destName: %v", destName)
 	}
@@ -380,11 +380,13 @@ func getClient() *github.Client {
 	return github.NewClient(httpClient)
 }
 
-func getRelease(client *github.Client, owner, repo, tag string) (*github.RepositoryRelease, error) {
+func getRelease(client *github.Client, owner, repo, tag string, pre bool) (*github.RepositoryRelease, error) {
 	ctx := context.Background()
 	var release *github.RepositoryRelease
 	var err error
-	if tag == "" || tag == "latest" {
+	if pre {
+		release, err = getLatestRelease(client, owner, repo)
+	} else if tag == "" || tag == "latest" {
 		release, _, err = client.Repositories.GetLatestRelease(ctx, owner, repo)
 	} else {
 		release, _, err = client.Repositories.GetReleaseByTag(ctx, owner, repo, tag)
@@ -395,6 +397,19 @@ func getRelease(client *github.Client, owner, repo, tag string) (*github.Reposit
 	return release, nil
 }
 
+func getLatestRelease(client *github.Client, owner, repo string) (*github.RepositoryRelease, error) {
+	ctx := context.Background()
+	releases, _, err := client.Repositories.ListReleases(ctx, owner, repo, &github.ListOptions{PerPage: 1})
+	if err != nil {
+		return nil, err
+	}
+
+	if len(releases) > 0 {
+		return releases[0], nil
+	}
+	return nil, nil
+}
+
 func ensureWinExt(destName string) string {
 	if runtime.GOOS == "windows" {
 		if !strings.HasSuffix(destName, ".exe") {

cmd/remove.go

@@ -20,8 +20,8 @@ var removeCmd = &cobra.Command{
 	Run: func(cmd *cobra.Command, args []string) {
 		binPath := viper.GetString("bin")
 		log.Debug("removeCmd", "args", args, "binPath", binPath)
-		//noConfirm, _ := cmd.Flags().GetBool("yes")
-		//log.Debug("Flags", "noConfirm", noConfirm)
+		//skipPrompts := viper.GetBool("yes")
+		//log.Info("Flags:", "skipPrompts", skipPrompts)
 
 		var name string
 		var err error

cmd/root.go

@@ -36,12 +36,18 @@ func Execute() {
 
 func init() {
 	cobra.OnInitialize(onInitialize)
-	rootCmd.Flags().StringP("asset", "a", "", "asset name to download")
-	rootCmd.Flags().StringP("name", "n", "", "binary file name to use")
-	rootCmd.Flags().BoolP("yes", "y", false, "answer yes to prompts")
+	rootCmd.PersistentFlags().StringP("asset", "a", "", "asset name to download")
+	_ = viper.BindPFlag("asset", rootCmd.PersistentFlags().Lookup("asset"))
+	rootCmd.PersistentFlags().StringP("name", "n", "", "binary file name to use")
+	_ = viper.BindPFlag("name", rootCmd.PersistentFlags().Lookup("name"))
+	rootCmd.PersistentFlags().BoolP("pre", "p", false, "include pre-releases")
+
+	rootCmd.PersistentFlags().BoolP("yes", "y", false, "answer yes to prompts")
+	_ = viper.BindPFlag("yes", rootCmd.PersistentFlags().Lookup("yes"))
 
 	rootCmd.PersistentFlags().StringP("bin", "b", "", "bin path to use")
 	_ = viper.BindPFlag("bin", rootCmd.PersistentFlags().Lookup("bin"))
+
 	rootCmd.PersistentFlags().CountVarP(&verbose, "verbose", "v", "verbose output (-vvv debug)")
 
 	rootCmd.Flags().BoolP("version", "V", false, "show installed version")
@@ -52,9 +58,13 @@ func onInitialize() {
 	log.Info("Log Level", "verbose", verbose)
 
 	configName := "install-release"
-	//viper.SetEnvPrefix("ir")
+	configType := "yaml"
 	viper.SetConfigName(configName)
-	viper.SetConfigType("yaml")
+	viper.SetConfigType(configType)
+	viper.SetEnvPrefix("ir")
+	viper.AutomaticEnv()
+
+	viper.SetDefault("yes", false)
 
 	homeDir, err := os.UserHomeDir()
 	if err != nil {
@@ -76,7 +86,7 @@ func onInitialize() {
 		if err := os.MkdirAll(configPath, 0755); err != nil {
 			log.Fatalf("Creating config directory: %v: %v", configPath, err)
 		}
-		configFile := filepath.Join(configPath, configName+".yaml")
+		configFile := filepath.Join(configPath, fmt.Sprintf("%s.%s", configName, configType))
 		log.Infof("Config File: %v", configFile)
 		viper.SetConfigFile(configFile)
 		if err := viper.SafeWriteConfigAs(configFile); err != nil {

docs/index.md

@@ -59,7 +59,13 @@ Install the latest release.
 ir owner/repo
 ```
 
-Install a specific version.
+Include pre-releases.
+
+```shell
+ir owner/repo --pre
+```
+
+Install a specific version/tag.
 
 ```shell
 ir owner/repo v4.2.0
@@ -97,10 +103,22 @@ ir list
 
 Remove an app.
 
+```shell
+ir remove
+```
+
+Remove an app by name.
+
 ```shell
 ir remove name
 ```
 
+Edit the settings.
+
+```shell
+ir config
+```
+
  
 
 !!! question

internal/pathmgr/pathmgr.go

@@ -12,7 +12,7 @@ func IsDirInPath(dirPath string) (found bool, err error) {
 	log.Debug("IsDirInPath", "dirPath", dirPath)
 
 	// NOTE: Determine how to handle PATH in DOCKER
-	if os.Getenv("DOCKER") != "true" {
+	if os.Getenv("DOCKER") == "true" {
 		log.Infof("IsDirInPath is Disabled in DOCKER")
 		return true, nil
 	}