Change cli

[TechMagister]

Move from OptionParser to Cli for better integration

class Watch < Sentry::SentryCommand
      command_name "watch"

      def run
        options.watch << "./config/**/*.cr"
        super
      end
end

[samueleaton]

When someone runs the install.rb script to install the cli tool it is copying the src files. You are adding a file to the src directory so you will probably need to update install.rb.

[samueleaton]

I believe it is a Crystal standard to capitalize acronyms (e.g. CLI, HTTP, JSON)

[TechMagister]

True, but I don't know if @mosop will change the name now :/

[samueleaton]

Also, the current CLI installation process does not use shards or dependencies, so we will need to figure that out. Correct me if I'm wrong, but I don't believe shards has support for installing CLI tools as a dependency.

[TechMagister]

I updated the install.rb file to take in account the modifications. ( I completely forgot it 😝 )

[samueleaton]

Seeing as this will introduce a dependency, is there a way to install Sentry while installing dependencies at the same time? Does shards support that feature? I like what you did to install.rb btw.

[samueleaton]

Also they way you are loading the app name in the CLI dynamically is a no brainer. 👍 That will be much better. Glad I could get some other eyes on this project to help button it up.

I would however make the DEFAULT_NAME start as nil and print an error if it is not set manually or found in shard.yml.

[jwaldrip]

You may want to look at Admiral as well: https://github.com/jwaldrip/admiral.cr