Duplicate uv --app message improvement

Open stefanofusai opened this issue 1 year ago • 3 comments

I've noticed (OCD...) that when doing uv --app <project>, when <project> already exists, the error message could be improved.

Currently, this is what is output:

This could be changed to error: App is already initialized in ...

Unfortunately I'm not well versed in Rust so I can't submit a PR, but the relevant code is here: https://github.com/astral-sh/uv/blob/main/crates/uv/src/commands/project/init.rs#L105

Let me know your thoughts!

Dec 13 '24 20:12 stefanofusai

I actually think this should remain "Project" — it's a general error message. --app is just a template for the project. The same error would apply for --lib.

Dec 13 '24 21:12 zanieb

You're right in that the Initialized project message should also be changed, and probably n other references

Dec 13 '24 21:12 stefanofusai

What is gained by switching from "Initialized project" to "Initialized application" and "Initialized library"?

Dec 13 '24 21:12 zanieb