Commit e05d13b
committed
Make the CLI robust at its edges
Config saving round-trips any input, the update check understands
pre-releases, non-interactive runs fail safely on a dirty tree, setup and
test regain the scriptability and filtering users had before, and every
command has CLI-level test coverage.
- save_config uses a TOML writer, so quotes and backslashes no longer
produce a config the tool cannot read back; load errors name the path
- version comparison via PEP 440, so beta users are told about newer betas
- dirty tree aborts in non-interactive mode; --allow-dirty opts back in
- setup takes --data, --data-file and --defaults like create/add
- test -t NAME / -s TARGET pass through to the invoke task and the command
exits with the test run's exit code
- serve/test/debug explain a missing uv, tasks.py, invoke or pytest
- auto-commit failures go to the styled error channel
- fixes "completion <shell> --install", which the new tests caught
Requires the matching test task in copier-templates; projects with an older
tasks.py get a message naming the fix.1 parent 4f6576d commit e05d13b
23 files changed
Lines changed: 1811 additions & 174 deletions
File tree
- plonecli
- tests
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
6 | | - | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
7 | 54 | | |
8 | 55 | | |
9 | 56 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
132 | 132 | | |
133 | 133 | | |
134 | 134 | | |
135 | | - | |
136 | | - | |
137 | | - | |
138 | | - | |
139 | | - | |
140 | | - | |
141 | | - | |
142 | | - | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
143 | 145 | | |
144 | 146 | | |
145 | 147 | | |
146 | | - | |
| 148 | + | |
147 | 149 | | |
148 | 150 | | |
149 | 151 | | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
150 | 178 | | |
151 | 179 | | |
152 | 180 | | |
| |||
205 | 233 | | |
206 | 234 | | |
207 | 235 | | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
208 | 250 | | |
209 | 251 | | |
210 | 252 | | |
| |||
314 | 356 | | |
315 | 357 | | |
316 | 358 | | |
| 359 | + | |
| 360 | + | |
| 361 | + | |
| 362 | + | |
317 | 363 | | |
318 | 364 | | |
319 | 365 | | |
| |||
0 commit comments