refactor: make categories optional in core config #437
Conversation
matejchalk
left a comment
There was a problem hiding this comment.
We could also consider having some prompt to create categories in place of the missing category sections in report.md or stdout summary 🤔 But that's a different discussion and out of scope of this PR, I believe.
packages/cli/src/lib/implementation/core-config.integration.test.ts
Outdated
Show resolved
Hide resolved
packages/cli/src/lib/implementation/core-config.integration.test.ts
Outdated
Show resolved
Hide resolved
d712c32 to
02761cd
Compare
@matejchalk |
02761cd to
f293555
Compare
f293555 to
068b1eb
Compare
068b1eb to
aae11ed
Compare
Done ✔️ |
BioPhoton
left a comment
There was a problem hiding this comment.
You could update the snippets in 'docs/custom-plugind.md'
Closes #407
I aimed for handling the optional categories only in the cli layer and maintained the existing data structure fully in reports. I just hide the sections for empty categories.
Follow-up by @Tlacenka:
undefinedand[]by configuring categories schema to allow empty array.