Repository navigation
fix(examples): Pass octane e2e on octane 0.12 - #6630
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing (37)
✨ Finishing Touches
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 |
|
View your CI Pipeline Execution ↗ for commit 245c35a
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version Preview1 package(s) bumped directly, 0 bumped as dependents. 🟨 Minor bumps
|
Fixes the octane e2e failures on
mainafter #6627 (failed run).colspan/rowspaninstead ofcolSpan/rowSpan. Octane 0.12's DEV diagnostics don't listcolSpanorrowSpanas known DOM props, so every header row logged "Octane does not recognize thecolSpanprop", which fails the smoke tests' console-error checks. Octane accepts the native lowercase names. The test fixture inoctane-tablegets the same change.basic-external-atomsandbasic-external-statesmoke tests wait one microtask after a scripted click before reading the DOM. Octane 0.12 commits a click's updates in the microtask after the event.All 33 octane examples pass
test:e2elocally (137 tests).Notes
React accepts
colSpan, so the missing entry is probably an Octane bug; it isn't in octane'smaineither. Once Octane fixes it, the camelCase spelling would work again.🤖 Generated with Claude Code
Summary by CodeRabbit
Documentation
colspanandrowspanattributes.Bug Fixes