Commit 6d07038
authored
fix(cli): import yaml lazily so the CLI works without the yaml extra (#4929)
Import yaml lazily in the schema CLI command
The module-level 'from yaml import dump' in litestar/cli/commands/schema.py
is imported eagerly via litestar.cli.main, so every CLI invocation failed
with ModuleNotFoundError when litestar was installed without the yaml extra
(pyyaml only comes in transitively with the standard extras via uvicorn).
Import yaml inside the YAML export branch and raise
MissingDependencyException('pyyaml', extra='yaml') if it is missing, matching
how other optional dependencies are handled.
Fixes #4449
Co-authored-by: uttam12331 <uttam12331@users.noreply.github.com>1 parent 1aa4aa5 commit 6d07038
2 files changed
Lines changed: 27 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
14 | | - | |
15 | 13 | | |
16 | 14 | | |
17 | 15 | | |
18 | 16 | | |
19 | 17 | | |
| 18 | + | |
20 | 19 | | |
21 | 20 | | |
22 | 21 | | |
| |||
31 | 30 | | |
32 | 31 | | |
33 | 32 | | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
34 | 40 | | |
35 | 41 | | |
36 | 42 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
103 | 103 | | |
104 | 104 | | |
105 | 105 | | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
0 commit comments