Draft: add --strict check
closes #3 (closed), replaces !10 (closed)
Merge request reports
Activity
requested review from @jkuchar1
assigned to @jpudil
Sémanticky to navazuje na změny, které jsem dělal v outputu a v error handlingu – scaffolder s touhle úpravou co nejvíc granulárně chytá pokémony, takže při každém spuštění projde, zkontroluje a vygeneruje úplně všechny definice.
Název přepínače je určitě otevřen diskuzi, taky si tím nejsem vůbec jistý. Ale bude potřeba najít vhodný kompromis mezi stručností (
--strict
) a srozumitelností (--fail-on-changes-that-would-be-lost
).Ta myšlenka byla původně jiná. Spíše něco ve smyslu
scaffolder check-consistency
. Tj. že to vůbec není pro uživatele, který si omylem něco přepíše, ale pro CI, kde to jednoduše failne a řekne, které soubory jsou nekonsistentní sdefinition
files.Myšlenka, že mi scaffolder řekne, v kterých souborech jsem dělal ruční změny a budou smazány mi přijde super, ale myslím, že to tím pádem vyžaduje uložit nějaký checksum, který teď nikde nemáme. :) Možná raději zavolej a domluvíme se, co s tím. :)
359 360 public function addDefinition(DefinitionResult $definitionResult): void 361 { 362 $this->definitions[] = $definitionResult; 363 } 364 365 /** 366 * @return DefinitionResult[] 367 */ 368 public function getDefinitions(): array 369 { 370 return $this->definitions; 371 } 372 } 373 374 final class DefinitionResult 314 return; 315 } 316 317 if (\is_string($exceptionFile)) { 318 $cwd = \getcwd(); 319 \assert(\is_string($cwd)); 320 321 $output->writeln(\sprintf( 322 'Error was logged in %s', 323 Path::makeRelative($exceptionFile, $cwd), 324 )); 325 } 326 } 327 } 328 329 final class FileResult mentioned in merge request !52 (merged)
added 2 commits
- 99d94888 - make result classes internal
- b0de230e - Add --strict option to fail if a generated file has changed in filesystem (closes #3 (closed), !10 (closed))
Zavírám, toto není správná cesta, ta jest prokopána v !53 (merged)