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

scripts: auto-generate MAVLink messages files #8

Merged

Conversation

ES-Alexander
Copy link
Contributor

@ES-Alexander ES-Alexander commented Jun 12, 2024

@patrickelectric I believe you're most relevant to review this :-)

@ES-Alexander ES-Alexander force-pushed the auto-generate-mavlink-messages branch from 26e780a to 9f34981 Compare November 6, 2024 11:07
@HTRamsey
Copy link

HTRamsey commented Jan 9, 2025

ping

@patrickelectric
Copy link
Member

@HTRamsey we are waiting for: ArduPilot/ardupilot#27226

@ES-Alexander
Copy link
Contributor Author

ES-Alexander commented Jan 9, 2025

@patrickelectric as above,

Ideally we'll avoid merging this PR until that one is merged, but I've implemented a hacky workaround in the meantime 🤷‍♂️

we can merge as-is if we don’t want to keep waiting, we’ll just need to remember to revert the commit to fix the script link between when the ArduPilot PR gets merged and I delete my branch :-)

@patrickelectric
Copy link
Member

Let's merge.. we can't wait forever for the PR to get merged

@ES-Alexander
Copy link
Contributor Author

If you’re happy with it, feel free to merge - I don’t have permissions in this repo.

I did test the functionality in November, so as far as I’m aware it should work as-is :-)

@patrickelectric patrickelectric merged commit f74ddcd into ArduPilot:main Jan 9, 2025
@ES-Alexander ES-Alexander deleted the auto-generate-mavlink-messages branch January 9, 2025 22:27
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.

3 participants