Skip to content

feat: Make better error message for invalid --app string#3977

Merged
provinzkraut merged 5 commits intolitestar-org:mainfrom
RenameMe1:main
Feb 2, 2025
Merged

feat: Make better error message for invalid --app string#3977
provinzkraut merged 5 commits intolitestar-org:mainfrom
RenameMe1:main

Conversation

@RenameMe1
Copy link
Copy Markdown
Contributor

Description

• I added new func that output readable errors when --app arg take invalid argument and stop process

I will be glad to receive feedback 😊.

Closes

Closes #3893

@RenameMe1 RenameMe1 requested review from a team as code owners January 27, 2025 15:28
@github-actions github-actions bot added area/private-api This PR involves changes to the privatized API size: small type/feat pr/external Triage Required 🏥 This requires triage labels Jan 27, 2025
@euri10
Copy link
Copy Markdown
Contributor

euri10 commented Jan 27, 2025

related: #3895

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.35%. Comparing base (b0322d5) to head (39a90e9).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3977   +/-   ##
=======================================
  Coverage   98.34%   98.35%           
=======================================
  Files         347      347           
  Lines       15756    15764    +8     
  Branches     1741     1741           
=======================================
+ Hits        15496    15504    +8     
  Misses        124      124           
  Partials      136      136           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@provinzkraut
Copy link
Copy Markdown
Member

@RenameMe1 Can you address the missing coverage?

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 2, 2025

Documentation preview will be available shortly at https://litestar-org.github.io/litestar-docs-preview/3977

@provinzkraut provinzkraut merged commit ecfe5c7 into litestar-org:main Feb 2, 2025
@RenameMe1
Copy link
Copy Markdown
Contributor Author

@all-contributors
please add @RenameMe1 for code.
please add @RenameMe1 for tests.

@allcontributors
Copy link
Copy Markdown
Contributor

@RenameMe1

I've put up a pull request to add @RenameMe1! 🎉

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

Labels

area/private-api This PR involves changes to the privatized API pr/external size: small Triage Required 🏥 This requires triage type/feat

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ehancement: CLI - Better error message for invalid --app string

4 participants