Skip to content

Commit 7a1a829

Browse files
committed
chore(tidy): clang-format APDS9999, add skill postRun hook.
1 parent 1442a52 commit 7a1a829

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

.agents/skills/add-sensor-component-v1/SKILL.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,11 @@ description: >
77
WipperSnapper — even if they just mention a sensor name and "wippersnapper" in the same breath.
88
Covers the full workflow: driver code, registration, component definition JSON, library deps,
99
build verification, clang-format, and PR creation for both repos.
10+
postRun: >
11+
Automatically use run_in_terminal to execute `clang-format -i` on the modified files,
12+
and execute `doxygen` (e.g., `doxygen Doxyfile`) to validate documentation blocks.
13+
Review the terminal output, identify any warnings or errors on the modified files,
14+
and apply fixes before continuing. If you have issues then you may want to clone adafruit/ci-arduino and then examine the CI scripts to understand the exact commands and checks used for clang-format and doxygen in the CI pipeline.
1015
---
1116

1217
# Add I2C Sensor Component to WipperSnapper v1

src/components/i2c/WipperSnapper_I2C.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -240,8 +240,7 @@ bool WipperSnapper_Component_I2C::initI2CDevice(
240240
_ahtx0->configureDriver(msgDeviceInitReq);
241241
drivers.push_back(_ahtx0);
242242
} else if (strcmp("apds9999", msgDeviceInitReq->i2c_device_name) == 0) {
243-
_apds9999 =
244-
new WipperSnapper_I2C_Driver_APDS9999(this->_i2c, i2cAddress);
243+
_apds9999 = new WipperSnapper_I2C_Driver_APDS9999(this->_i2c, i2cAddress);
245244
if (!_apds9999->begin()) {
246245
WS_DEBUG_PRINTLN("ERROR: Failed to initialize APDS9999!");
247246
_busStatusResponse =

0 commit comments

Comments
 (0)