Skip to content

Conversation

@AliYmn
Copy link

@AliYmn AliYmn commented Nov 24, 2024

Improve Error Messages in FastAPI CLI

Enhanced error messages to provide better guidance for common issues:

Changes

  • Added detailed file path suggestions for default app discovery
  • Improved FastAPI installation instructions with pip commands
  • Made error messages more actionable and user-friendly

@AliYmn AliYmn closed this Nov 24, 2024
@AliYmn AliYmn reopened this Nov 24, 2024
@AliYmn AliYmn changed the title 👷 Improve error handling for FastAPI import and default app file dete… 👷 Improve error messages and exception handling Nov 24, 2024
@AliYmn
Copy link
Author

AliYmn commented Nov 24, 2024

@tiangolo I believe this is my initial straightforward request for assistance. It would bring me great joy if you could give it some thought! 😄

@tiangolo
Copy link
Member

tiangolo commented Dec 3, 2024

Thanks for the interest, but the logic there is already as intended, e.g. the exception at the top is not supposed to be raised, that is checked somewhere else in the code.

The best approach would be to use the tool, and when there's something that you would want improved, try the changes locally and see if they do what you expect.

Sadly I also suspect it's AI-generated. 😔

For now, I'll pass on this one. ☕

@tiangolo tiangolo closed this Dec 3, 2024
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.

2 participants