The GitHub Actions job "Backport Checks" on 
texera.git/fix/8283-filled-area-plot-line-group has succeeded.
Run started by GitHub user kz930 (triggered by kz930).

Head commit for run:
0a0093af3e5f0fbfe2835fb84b6681f637b9206d / kary zheng <[email protected]>
feat(visualization): require the Filled Area Plot line group under its switch

Turning on Split Plot by Line Group makes the line group required, because
code generation asserts it and the run ends on `Line Group cannot be empty`.
The field is declared optional and carried no conditional constraint, so the
property panel accepted the configuration and the error waited for the run.
A field that is optional is exactly the one a user leaves behind.

The schema states the rule instead, in the conditional form the Sklearn text
columns already use: required under `facetColumn`, so the panel refuses the
configuration while it is being written. Conditional rather than a plain
required, so a freshly dropped operator, whose switch is off, is not flagged
for a field it has no use for.

The assertion stays as the last line of defence, and the tests that pin it
stay with it.

Co-Authored-By: Claude Opus 5 (1M context) <[email protected]>

Report URL: https://github.com/apache/texera/actions/runs/33374645406

With regards,
GitHub Actions via GitBox

Reply via email to