Skip to content

Add ACCT to Manufacturers list - #1097

Open
RunnyCow wants to merge 1 commit into
betaflight:masterfrom
RunnyCow:Branch_master
Open

Add ACCT to Manufacturers list#1097
RunnyCow wants to merge 1 commit into
betaflight:masterfrom
RunnyCow:Branch_master

Conversation

@RunnyCow

@RunnyCow RunnyCow commented May 7, 2026

Copy link
Copy Markdown

Description

This PR is a documentation-only update to the official Betaflight manufacturer ID list (Manufacturers.md). It adds a new manufacturer ID entry for Accton Technology Corporation (ACCT) to support workflows in the Betaflight Configurator.

  • Added Entry: ACCT (Accton Technology Corporation)
  • Contact URL: https://www.accton-iot.com
  • Placement: The new row has been inserted alphabetically between LEGA and AEDR to maintain order and file consistency.
  • Impact: No existing manufacturer entries or functional codes were modified or removed.

Contribution Type

  • Documentation / Website / Wiki / Translation
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Hardware Compliance & Review Requirements

As this is a pure documentation asset change under the betaflight/config guidelines, full functional hardware validation and flight testing do not apply to this specific pull request.

  • Hardware Testing Status: N/A (No target firmware or hardware configuration profiles are introduced)
  • Applicable Guidelines: This pull request follows the standard peer-review and data accuracy validation process for Manufacturers.md list additions, rather than the functional hardware certification workflow required for new flight controller targets.

Checklist

  • My pull request is from a custom branch, not direct from master
  • Tested on hardware (N/A - Documentation only)
  • Follows coding style (N/A)
  • Updated documentation / manufacturer list
  • Verified target compilation (N/A)

Summary by CodeRabbit

  • Documentation
    • Added Accton Technology Corporation to the manufacturer reference table.
    • Included manufacturer ID ACCT and its contact website.

@coderabbitai

coderabbitai Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 9c2c1d49-ebdf-4330-94ca-cc190ceac55c

📥 Commits

Reviewing files that changed from the base of the PR and between d336430 and ede90e7.

📒 Files selected for processing (1)
  • Manufacturers.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • Manufacturers.md

Walkthrough

The manufacturer registry now includes manufacturer id ACCT, Accton Technology Corporation, and its contact URL.

Changes

Manufacturer Registry Update

Layer / File(s) Summary
Manufacturer entry
Manufacturers.md
Adds ACCT for Accton Technology Corporation with its contact URL.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the addition of the ACCT manufacturer entry.
Description check ✅ Passed The description explains the documentation change, its scope, placement, impact, and why hardware checks do not apply.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

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

@RunnyCow

Copy link
Copy Markdown
Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented May 12, 2026

Copy link
Copy Markdown
Contributor
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@RunnyCow

Copy link
Copy Markdown
Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented May 12, 2026

Copy link
Copy Markdown
Contributor
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@ot0tot

ot0tot commented May 12, 2026

Copy link
Copy Markdown
Contributor

We should wait to merge new manufacturers until they have an approved target under review as well

@sugaarK

sugaarK commented May 29, 2026

Copy link
Copy Markdown
Member

We should wait to merge new manufacturers until they have an approved target under review as well

agree

@sugaarK sugaarK added the Contact Betaflight Team cloudtargets@betaflight.com label May 29, 2026
@RunnyCow RunnyCow closed this Jul 31, 2026
@RunnyCow RunnyCow reopened this Jul 31, 2026
@RunnyCow RunnyCow closed this Jul 31, 2026
@RunnyCow RunnyCow reopened this Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Contact Betaflight Team cloudtargets@betaflight.com

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants