-
Notifications
You must be signed in to change notification settings - Fork 18k
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
Add EchoPilot AI board id #23653
Conversation
There was a problem hiding this 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 |
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
try 1408
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. |
@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! |
No description provided.