Skip to content

Move more business logic out of the GUI#997

Merged
amilcarlucas merged 2 commits into
masterfrom
refactor_param_editor
Oct 30, 2025
Merged

Move more business logic out of the GUI#997
amilcarlucas merged 2 commits into
masterfrom
refactor_param_editor

Conversation

@amilcarlucas

Copy link
Copy Markdown
Collaborator

Make some functions protected to minimize the public API

Make some functions protected to minimize the public API
Refactored all freedesktop.org code to a single file
@github-actions

Copy link
Copy Markdown
Contributor

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
8897 7514 84% 80% 🟢

New Files

File Coverage Status
ardupilot_methodic_configurator/backend_filesystem_freedesktop.py 67% 🟢
TOTAL 67% 🟢

Modified Files

File Coverage Status
ardupilot_methodic_configurator/main.py 81% 🟢
ardupilot_methodic_configurator/backend_filesystem_program_settings.py 99% 🟢
ardupilot_methodic_configurator/configuration_manager.py 80% 🟢
ardupilot_methodic_configurator/frontend_tkinter_base_window.py 94% 🟢
ardupilot_methodic_configurator/frontend_tkinter_parameter_editor.py 82% 🟢
ardupilot_methodic_configurator/frontend_tkinter_show.py 90% 🟢
TOTAL 88% 🟢

updated for commit: 058b196 by action🐍

@github-actions

Copy link
Copy Markdown
Contributor

Test Results

    3 files      3 suites   2m 58s ⏱️
2 008 tests 2 006 ✅ 2 💤 0 ❌
6 024 runs  6 018 ✅ 6 💤 0 ❌

Results for commit 058b196.

@amilcarlucas amilcarlucas merged commit 0e0c54c into master Oct 30, 2025
31 checks passed
@amilcarlucas amilcarlucas deleted the refactor_param_editor branch October 30, 2025 01:15
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.

1 participant