[API] Restore public CLI handler compatibility imports - #1049
podkidyshev wants to merge 3 commits into
Conversation
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. 🧰 Additional context used📚 Code guidelines (1)No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository: NVIDIA/cloudai/.coderabbit.yaml Review profile: ASSERTIVE Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe CLI handlers module adds deprecated wrappers for 19 handler functions. Each wrapper emits a ChangesCLI handler compatibility
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The compatibility module restores legacy handler imports with deprecation warnings while preserving handler behavior. No actionable merge-blocking risk is established; merge after normal checks pass. 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
64bc471 to
ac72903
Compare
ac72903 to
00eb42e
Compare
00eb42e to
fd55bb0
Compare
Signed-off-by: Ivan Podkidyshev <ipodkidyshev@nvidia.com>
Signed-off-by: Ivan Podkidyshev <ipodkidyshev@nvidia.com>
Signed-off-by: Ivan Podkidyshev <ipodkidyshev@nvidia.com>
7b08159 to
0a24d07
Compare
Summary
cloudai.cli.handlersimports as deprecated forwarding wrappers after the handler-module rename in [API] Move shared handlers out of the CLI package #1046.DeprecationWarningat the caller.Test Plan
Additional Notes
This PR changes only the compatibility module. Follow-ups: #1063 (CLI config/listing), #1064 (CLI execution), #1060 (Path API), #1061 (TOML text), and #1062 (background execution).