Skip to content

Set t5s3_epaper_inkhud to extra#10037

Merged
thebentern merged 1 commit into
meshtastic:masterfrom
vidplace7:disable-t5s3_epaper_inkhud
Mar 30, 2026
Merged

Set t5s3_epaper_inkhud to extra#10037
thebentern merged 1 commit into
meshtastic:masterfrom
vidplace7:disable-t5s3_epaper_inkhud

Conversation

@vidplace7

Copy link
Copy Markdown
Member

t5s3_epaper_inkhud is incomplete (missing macros). Disable it for now.

Also fixes the board_check evaluation in generate_ci_matrix.py so that false actually works correctly.

t5s3_epaper_inkhud is incomplete (missing macros). Disable it for now.

Also fixes the board_check evaluation so that `false` actually works.
@github-actions github-actions Bot added the bugfix Pull request that fixes bugs label Mar 30, 2026
@vidplace7 vidplace7 added the cleanup Code cleanup or refactor label Mar 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adjusts CI/build selection so an incomplete ESP32-S3 InkHUD variant is excluded from normal CI builds, and fixes CI matrix generation so board_check = false is honored correctly.

Changes:

  • Mark t5s3_epaper_inkhud as an extra-level board and disable its board_check.
  • Fix generate_ci_matrix.py boolean parsing for board_check so "false" no longer evaluates as True.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
variants/esp32s3/t5s3_epaper/platformio.ini Sets the InkHUD environment to board_level = extra and board_check = false to keep it out of standard CI.
bin/generate_ci_matrix.py Corrects board_check parsing by comparing the string value to "true" rather than using Python bool() on a non-empty string.

@thebentern thebentern merged commit 33d526f into meshtastic:master Mar 30, 2026
84 of 85 checks passed
@vidplace7 vidplace7 deleted the disable-t5s3_epaper_inkhud branch March 30, 2026 18:56
skrashevich pushed a commit to skrashevich/meshtastic-firmware that referenced this pull request Apr 4, 2026
t5s3_epaper_inkhud is incomplete (missing macros). Disable it for now.

Also fixes the board_check evaluation so that `false` actually works.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Pull request that fixes bugs cleanup Code cleanup or refactor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants