Skip to content

Conversation

dakejahl
Copy link
Contributor

INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
INFO [mavlink] Upgrade to MAVLink v2 because of incoming packet
from #25583

@dakejahl dakejahl requested a review from MaEtUgR October 14, 2025 08:16
@dakejahl dakejahl closed this Oct 14, 2025
@dakejahl
Copy link
Contributor Author

replaced by #25758

@MaEtUgR MaEtUgR reopened this Oct 15, 2025
@MaEtUgR MaEtUgR changed the title mavlink: stop version upgrade spam mavlink: refactorings from fixing the message spam Oct 15, 2025
MaEtUgR
MaEtUgR previously approved these changes Oct 15, 2025
Copy link
Member

@MaEtUgR MaEtUgR left a comment

Choose a reason for hiding this comment

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

@dakejahl I'm so sorry I didn't see your fix when making #25758 I'll change my notifications setup.

Still I think it's worth to bring in your refactoring hence I rebased it and extended a bit according to my test. What do you think?

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.

2 participants