sea as an alternative bin name to sea-orm-cli

[ZhangHanDong]

I feel that the command sea-orm-cli is too long and inconvenient to use, so it should be more convenient to simplify it to sea now.

What do you think?

Changes

The sea command is now an option, and a new bin file has been added to achieve this.

The only downside: bin/sea.rs is a duplicate of bin/main.rs, but it has now been modified to have only one main function, so if there are changes, they are simply copied to sea.rs. It won't be too much trouble to maintain.

Now cargo run still defaults to sea-orm-cli

Install and Usage:

> cargo install sea-orm-cli 
> sea-orm-cli help

Or:

> cargo install --bin sea
> sea help

[tyt2y3]

I think this is a reasonable thought. sea-orm-cli is more canonical, but sea is a perfectly fine shorthand. Is it possible to make it an alias?

[ZhangHanDong]

I think this is a reasonable thought. sea-orm-cli is more canonical, but sea is a perfectly fine shorthand. Is it possible to make it an alias?

It doesn't seem to be possible to configure aliases in the toml file, but you can configure multiple bins like this, which will generate two main target files：

[[bin]]
name = "sea-orm-cli"
path = "src/main.rs"

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

it's ok?

[tyt2y3]

That's interesting! Certainly better than a breaking name change. I also think we need to set the default binary, other cargo run will confuse

[ZhangHanDong]

That's interesting! Certainly better than a breaking name change. I also think we need to set the default binary, other cargo run will confuse

This has the minor drawback of reporting a compile warning at compile time:

warning: /Projects/sea-orm/sea-orm-cli/Cargo.toml: file found to be present in multiple build targets

[ZhangHanDong]

@tyt2y3

The sea command is now an option, and a new bin file has been added to achieve this.

The only downside: bin/sea.rs is a duplicate of bin/main.rs, but it has now been modified to have only one main function, so if there are changes, they are simply copied to sea.rs. It won't be too much trouble to maintain.

Now cargo run still defaults to sea-orm-cli

Install and Usage:

> cargo install sea-orm-cli 
> sea-orm-cli help

Or:

> cargo install --bin sea
> sea help

[ZhangHanDong]

@tyt2y3 what do you think?

[tyt2y3]

Awesome

[ZhangHanDong]

@tyt2y3 Fixed CI compile error

[ZhangHanDong]

@tyt2y3 need approve workflow

[ZhangHanDong]

@tyt2y3

Can't it be merged yet? Is there anything that needs to be changed?

[tyt2y3]

LGTM

[ZhangHanDong]

@tyt2y3

P.S Can't seaorm automatically generate migration files now? Except for the sea migrate init.

[billy1624]

@tyt2y3

P.S Can't seaorm automatically generate migration files now? Except for the sea migrate init.

Not supported at the moment. I think you want something like sea migrate new name_of_new_migration_file which create a new migration file?

[ZhangHanDong]

Yes. Like the rails migrate command