geido commented on PR #33180:
URL: https://github.com/apache/superset/pull/33180#issuecomment-5684830129

   > We might not need a whole new action for this. You could just add a few 
filenames to [this 
file](https://github.com/apache/superset/blob/master/.github/labeler.yml).
   
   Finally circling back on this — you were right, it's a handful of globs and 
the workflow is gone. Rebased on master.
   
   Couple of things worth your eye: I widened it past `plugins/`, since native 
filters and chart customizations register through the same plugin API from 
`src/` and their config gets persisted into dashboard `json_metadata`. And it 
only flags now, no CI gate — shout if you'd rather it had teeth, 
`no-hold-label.yml` is already the pattern for that. Happy to rename the label 
to a `viz:` one too if that sits better with the TODO at the top of that file.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to