Skip to content

Pr claude2/test board periph#49

Open
peterbarker wants to merge 4 commits into
masterfrom
pr-claude2/test-board-periph
Open

Pr claude2/test board periph#49
peterbarker wants to merge 4 commits into
masterfrom
pr-claude2/test-board-periph

Conversation

@peterbarker

Copy link
Copy Markdown
Owner

Summary

Classification & Testing (check all that apply and add your own)

  • Checked by a human programmer
  • Non-functional change
  • No-binary change
  • Infrastructure change (e.g. unit tests, helper scripts)
  • Automated test(s) verify changes (e.g. unit test, autotest)
  • Tested manually, description below (e.g. SITL)
  • Tested on hardware
  • Logs attached
  • Logs available on request

Description

peterbarker and others added 4 commits June 22, 2026 12:53
some checks went into check_branch_conventions.py instead.

Adds a comment so this doesn't happen again
An AP_Periph board with a README.md that has no image. AP_Periph boards
must ship a README but need not embed an image, so test_new_boards.py is
expected to accept it.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Drop CI workflows unrelated to verifying these new boards; the
Test-for-Malformed-Commits workflow runs test_new_boards.py.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Trigger on push (so the branch's own workflow file is used), drop the
ArduPilot-only job guard, and compare against ArduPilot master instead of
the pull_request base context.  Run test_new_boards.py even if the branch
conventions step fails, so the board result is always visible.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@peterbarker peterbarker force-pushed the pr-claude2/test-board-periph branch from 29a5932 to 4d1dabd Compare June 22, 2026 03:09
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.

1 participant