borgo icon indicating copy to clipboard operation
borgo copied to clipboard

Added `clap` crate for CLI

Open enrichman opened this issue 1 year ago • 2 comments

// TODO use a proper parsing lib

As per comment found in main this PR adds the clap crate to parse the arguments and create a more standard CLI.

Screen Shot 2024-05-13 at 18 18 26

The behaviour is the same as before, but in the future I would personally change/move the test command into some internal thing, instead of having it in the "main" binary.

It will be nice to have a borgo build, and eventually a borgo test that will execute some Borgo "test", moving the "compiler" test to something else. 😄 Since the project is named compiler that is the default name. borgo would be better.

enrichman avatar May 13 '24 16:05 enrichman

Since the project is named compiler that is the default name. borgo would be better.

My man, try smashing this into the Cargo.toml:

[[bin]]
name = "borgo"
path = "src/main.rs"

jmackie avatar May 15 '24 11:05 jmackie

Since the project is named compiler that is the default name. borgo would be better.

My man, try smashing this into the Cargo.toml:

[[bin]]
name = "borgo"
path = "src/main.rs"

Yep, I know, but thanks for pointing this out. I just didn't want to make this "breaking" change.

What's your thought about this @alpacaaa? Can we do this?

enrichman avatar May 15 '24 12:05 enrichman