Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add EchoPilot AI board id #23653

Closed
wants to merge 1 commit into from
Closed

Add EchoPilot AI board id #23653

wants to merge 1 commit into from

Conversation

climr
Copy link
Contributor

@climr climr commented May 1, 2023

No description provided.

Copy link
Contributor

@tridge tridge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry, you will need to change your board ID. We were negligent in not merging this earlier.

@@ -261,6 +261,7 @@ AP_HW_CUBEORANGE_PERIPH 1400
AP_HW_CUBEBLACK_PERIPH 1401
AP_HW_PIXRACER_PERIPH 1402
AP_HW_SWBOOMBOARD_PERIPH 1403
AP_HW_EchoPilotAI 1404
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

apologies, this is failing CI with:
ValueError: Duplicate ID 1404 in file for (AP_HW_EchoPilotAI) and (AP_HW_VIMDRONES_FLOW)

@@ -261,6 +261,7 @@ AP_HW_CUBEORANGE_PERIPH 1400
AP_HW_CUBEBLACK_PERIPH 1401
AP_HW_PIXRACER_PERIPH 1402
AP_HW_SWBOOMBOARD_PERIPH 1403
AP_HW_EchoPilotAI 1404
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try 1408

@rmackay9
Copy link
Contributor

It would be good to also modify the commit message to start with "Tools:" or "Bootloader:". To be clear, I meant the commit message, no the PR title.

@peterbarker
Copy link
Contributor

@climr please open a fresh pull request if you still want to reserve a board ID. We'll try to be a bit more diligent next time!

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.

5 participants