Skip to content

Conversation

@KenVanHoeylandt
Copy link
Contributor

@KenVanHoeylandt KenVanHoeylandt commented Jan 31, 2026

Summary by CodeRabbit

  • Chores
    • Standardized device module identifiers across all supported devices to use consistent naming conventions with lowercase alphanumeric and hyphen-separated formats.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 31, 2026

📝 Walkthrough

Walkthrough

The pull request updates the module name initialization across 40+ device driver modules. Each change modifies the .name field in the exported struct Module device_module instance from a generic "Module" identifier (or existing descriptive name) to a device-specific identifier in kebab-case format matching the directory name (e.g., "btt-panda-touch", "cyd-2432s024c", "m5stack-core2", etc.). The start and stop function pointers remain unchanged. Each file receives a single line modification.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Rename device modules' directly and clearly summarizes the main change across all 48 modified files—updating device_module.name initialization values to match their device identifiers.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch rename-device-modules

Comment @coderabbitai help to get the list of available commands and usage tips.

@KenVanHoeylandt KenVanHoeylandt merged commit 5993ceb into main Jan 31, 2026
1 of 13 checks passed
@KenVanHoeylandt KenVanHoeylandt deleted the rename-device-modules branch January 31, 2026 12:11
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