Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Organize main.rs differently in Rust projects #16

Closed
brson opened this issue Mar 26, 2019 · 2 comments
Closed

Organize main.rs differently in Rust projects #16

brson opened this issue Mar 26, 2019 · 2 comments
Labels
type/enhancement New feature or request

Comments

@brson
Copy link
Contributor

brson commented Mar 26, 2019

Per discussion in #10, the way I chose to to put the binary source at src/bin/main.rs is awkward because it produces a binary called main, which has to be explained. Putting that file somewhere else would do the right thing by default and avoid the subject.

Today I think it would be better to move this somewhere else, probably either src/bin/kvs.rs or src/main.rs. Need to decide which is the better practice. I'm inclined to go with the former, as putting lib.rs and main.rs in the same folder is a bit confusing, with the source for both intermingling.

@brson brson added the type/enhancement New feature or request label Mar 26, 2019
@brson brson added this to To do in Rust training mvp via automation Mar 26, 2019
@brson brson changed the title Organize main.rs differently Organize main.rs differently in Rust projects Mar 26, 2019
@sticnarf
Copy link
Collaborator

I agree it's better to just rename src/bin/main.rs to src/bin/kvs.rs. In case we need to add other executables, it looks better to place all executables in the same directory.

@brson
Copy link
Contributor Author

brson commented Mar 29, 2019

FIxed.

@brson brson closed this as completed Mar 29, 2019
Rust training mvp automation moved this from To do to Done Mar 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement New feature or request
Projects
No open projects
Development

No branches or pull requests

2 participants