Skip to content

S31 support - #69

Merged
KenVanHoeylandt merged 1 commit into
TactilityProject:mainfrom
Shadowtrance:s31-support
Nov 12, 2026
Merged

KenVanHoeylandt merged 1 commit into
TactilityProject:mainfrom
Shadowtrance:s31-support

Conversation

@Shadowtrance

@Shadowtrance Shadowtrance commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor

Summary by CodeRabbit

  • New Features
    • Added support for the ESP32-S31 platform across Air Mouse, Brainfuck, Breakout, Calculator, Diceware, ePub Reader, GPIO, Graphics Benchmark, Magic 8 Ball, Media Keys, Mystify Demo, Serial Console, Snake, TamaTac, Todo List, TwoEleven, and Web Server.

@coderabbitai

coderabbitai Bot commented Nov 12, 2026 •

Copy link
Copy Markdown

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

Seventeen app manifests add esp32s31 to their declared target platforms. Each manifest also increments its version name and version code.

Merge Risk: Minimal · up to 73385

This adds S31 platform support to the app manifests and bumps their versions. The default SDK index already includes the esp32s31 platform, and no concrete defect was found, so the change looks ready to merge.

Pre-merge checks: Passed 4

Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title "S31 support" matches the changeset. Every manifest adds esp32s31 to target.platforms and bumps the app version. The title names the main change, though it is brief and does not say that the…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Title check

Explanation

The title "S31 support" matches the changeset. Every manifest adds esp32s31 to target.platforms and bumps the app version. The title names the main change, though it is brief and does not say that the change applies to app manifests. The commit message matches the title.

Finishing Touches

Generate unit tests (beta)

  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

Share

Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9253598d-44af-417e-b96d-d8a91973e8b1
Commits

Reviewing files that changed from the base of the PR and between c108b23 and 73385f3.

Files selected for processing: 17

  • Apps/AirMouse/manifest.properties
  • Apps/Brainfuck/manifest.properties
  • Apps/Breakout/manifest.properties
  • Apps/Calculator/manifest.properties
  • Apps/Diceware/manifest.properties
  • Apps/EpubReader/manifest.properties
  • Apps/GPIO/manifest.properties
  • Apps/GfxBenchmark/manifest.properties
  • Apps/Magic8Ball/manifest.properties
  • Apps/MediaKeys/manifest.properties
  • Apps/MystifyDemo/manifest.properties
  • Apps/SerialConsole/manifest.properties
  • Apps/Snake/manifest.properties
  • Apps/TamaTac/manifest.properties
  • Apps/TodoList/manifest.properties
  • Apps/TwoEleven/manifest.properties
  • Apps/WebServer/manifest.properties

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

@KenVanHoeylandt
KenVanHoeylandt merged commit 4404876 into TactilityProject:main Nov 12, 2026
37 checks passed
@KenVanHoeylandt

Copy link
Copy Markdown
Contributor

Thanks again!

@Shadowtrance
Shadowtrance deleted the s31-support branch November 13, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants