Skip to content

Conversation

kornelski
Copy link
Contributor

Fixes #5431

@rust-highfive
Copy link

r? @matklad

(rust_highfive has picked a reviewer for you, use r? to override)

@matklad
Copy link
Contributor

matklad commented May 2, 2018

We've discussed this at the Cargo meeting, and the conclusion was that, while today src/lib.rs + src/main.rs is the default pattern for applications, we are actually not sure that this is the proper way to structure code, and not just a tradition and a historical accident.

So, we are not ready to promote this pattern even more, until we decide that this is indeed a good pattern to promote. The discussion about that happens at rust-lang/api-guidelines#167.

Let's keep this PR open, in case that that discussion ends quickly and in favor of the current pattern!

@alexcrichton
Copy link
Member

I'm gonna close this in favor of rust-lang/api-guidelines#167 for now as it's been awhile, but we can of course reopen if the discussion comes to a close!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants