I3eka commented on PR #45721:
URL: https://github.com/apache/superset/pull/45721#issuecomment-6435438730

   Reviewed the current discussion and the focused implementation, and merged 
Apache master `037abb7d0b433c1274ab55f95e5613310f132462` without rewriting the 
branch. Published head: `b0a5a4268a4fabd120852fdfdaedf5b5b3eedd96`. There are 
no inline review threads or outstanding substantive reviewer findings on this 
PR in this audit.
   
   - 698 tests passed: the full AI unit suite and the single-migration-head 
check.
   - 105 AI frontend tests passed on the identical full frontend Git tree 
`91c05bee9972b32d81248e1a3575e8f4e31edd75` in #44497; reused that result. 
Frontend type/lint hooks were rerun on this branch.
   - All applicable pre-commit hooks passed for the 116 files in this PR delta.
   - The translation template matches a fresh extraction from the exact 
committed tree.
   
   No data-specific column names, metric formulas or heuristics are embedded. 
Contract tests validate context availability; they do not prove that every 
model answer will use it correctly. The SDK 2 compatibility fix from #45723 is 
included because current Apache dependencies exposed an SDK 1-only test/client 
in this branch.
   
   Updated the description to match this head and its validation. This stays 
**Draft / ON HOLD** behind #42805 and SIP #42806. No deployment, live database 
migration or external provider call was performed; GitHub CI is separate from 
these local checks.


-- 
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