Conversation
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 30 minutes and 24 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
…e, ...) (#203) The STM32 orchestrator installs the STM32duino core but never walked libraries/*/ to surface bundled libraries to sketches. Any sketch that transitively included <SPI.h> (e.g. FastLED's STM32 fastspi path) failed with "fatal error: SPI.h: No such file or directory" despite SPI living right there in the cache. Mirror PR #164's Teensy fix: - Add a shared FrameworkLibrary model + discover_framework_libraries() walker in fbuild-packages, used by both TeensyCores and Stm32Cores. - Extract the #include-scanning / transitive-closure resolver out of the Teensy orchestrator into crate::framework_libs so both platforms share one code path. - Wire Stm32Cores::get_framework_libraries() + get_framework_library_include_dirs() into the STM32 orchestrator after SrcWrapper so the resolver picks up SPI/Wire/EEPROM/... on demand and their include dirs reach the sketch compile command. Fixes #202 Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Summary
Fixes #163.
PaulStoffregen/coresarchive to PlatformIO's fullframework-arduinoteensypackage.SPI.handOctoWS2811.hare visible to Teensy builds.SPI.handOctoWS2811.h.Verification
cargo test -p fbuild-packages library::teensy_corecargo test -p fbuild-build teensy::orchestratorcargo test -p fbuild-build --test teensy_build build_teensy41_blink -- --ignored --nocapturecargo test -p fbuild-build --test teensy_build build_teensy41_spi_octo_headers -- --ignored --nocapture