build: remove unused .zig codegen outputs#30915
Conversation
The codegen system generates .zig files as legacy artifacts from the Zig era. These files are no longer compiled or consumed by anything (the Rust build uses .rs counterparts). Removing them reduces codegen steps from 95 to 78 (-18%). Removed outputs: - ErrorCode.zig, ZigGeneratedClasses.zig - cpp.zig (companion to cpp.rs) - ci_info.zig (entire emitCiInfo step removed) - ResolvedSourceTag.zig, GeneratedBindings.zig, GeneratedJS2Native.zig
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
Walkthrough
ChangesRemove legacy Zig codegen artifacts
🚥 Pre-merge checks | ✅ 3 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Closing — this fix has been incorporated into PR #30917 (feat: add initial HarmonyOS support), which contains all OHOS changes in a single cohesive PR. |
|
Superseded by #30917 |
The codegen system generates .zig files alongside the actually-used .rs/.cpp/.h files. These .zig files are legacy artifacts from the Zig era - they are no longer compiled or consumed. Removing them reduces codegen steps from 95 to 78.