jaymasiwal commented on PR #39156: URL: https://github.com/apache/superset/pull/39156#issuecomment-4506144197
@rusackas Thanks! Regarding risk: replacing circular references with a "[Circular]" string placeholder is completely safe for downstream parsers. It serializes as a standard string rather than throwing a runtime error or silently omitting keys, keeping behavior transparent without breaking schemas. I've also addressed all of Copilot's feedback (gating console.warn to dev-only, cleaning test types, and fully DRYing the DeckGL plugin to a direct re-export). All CI checks are green and ready for your review! -- 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]
