Skip to content

Conversation

ThomasVestergaard
Copy link

@ThomasVestergaard ThomasVestergaard commented Sep 25, 2025

Documentation:

  • Added a new manufacturer ID entry: AGPT (Air Gapped Technologies ApS) with URL https://airgapped.dk/ to the official list.
  • Entry is added alphabetically, placing it as number 4 from the top.
  • No other changes made.

Summary by CodeRabbit

  • Documentation
    • Expanded the Manufacturer IDs reference by adding AGPT (Air Gapped Technologies) with its official URL, positioned after AEDR for consistency.
    • No removals or functional changes; this is a content update only.
    • Improves completeness, clarity, and ease of lookup for users consulting the manufacturer identifiers table.

Copy link
Contributor

coderabbitai bot commented Sep 25, 2025

Caution

Review failed

Failed to post review comments

Walkthrough

Added a single new manufacturer entry (AGPT — Air Gapped Technologies with URL) to the Manufacturer Ids table in Manufacturers.md after the AEDR row. No code, logic, or exported entities changed.

Changes

Cohort / File(s) Summary
Docs: Manufacturer IDs table
Manufacturers.md
Inserted a new table row for AGPT (Air Gapped Technologies) with its URL after AEDR; no deletions or other modifications.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested labels

New manufacturer

Suggested reviewers

  • ot0tot
  • nerdCopter

Pre-merge checks and finishing touches and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description Check ⚠️ Warning The PR description does not follow the repository’s required template because it omits the Pull-Request requirements, Hardware Compliance Requirements, Housekeeping section, and Checklist defined in the template. It only provides a brief note about the documentation change without addressing mandatory review or compliance details. Reviewers cannot verify adherence to repository standards with the current description. Please update the PR description to include all sections from the repository’s template—specifically the mandatory review process, hardware compliance requirements, housekeeping instructions, and a completed checklist—to ensure full alignment with submission guidelines.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title succinctly states that AGPT has been added as a manufacturer and directly reflects the modification made to the Manufacturers.md file. It is concise, specific, and free of extraneous detail. A reviewer scanning the PR list can immediately understand the primary change.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

@ot0tot
Copy link
Contributor

ot0tot commented Sep 25, 2025

Do you have a FC design under review currently? If you have not yet, please reach out to @sugaarK on the Betaflight Discord or email partnersprogram@betaflight.com

@ThomasVestergaard
Copy link
Author

Do you have a FC design under review currently? If you have not yet, please reach out to @sugaarK on the Betaflight Discord or email partnersprogram@betaflight.com

Not yet, but one will be underway soon. I'll reach out to @sugaarK when we're ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

New manufacturer Add manufacturer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants