feature/AddConfigMenu#9
Merged
Merged
Conversation
ilang-flytrex
commented
May 20, 2026
- Add config.xml configuration file
- Add 'Configurations' menu
- Add stale cx_Freeze cleaning to the deploy stage
- Add a functionality to select the DSDL branch for update
- Add a functionality that extract all the existing branches in 'Flytrex/public_regulated_data_types' repo
- Add the version to the main window
- Increment the flytrex version to 0.0.3
2. Add 'Configurations' menu 3. Add stale cx_Freeze cleaning to the deploy stage 4. Add a functionality to select the DSDL branch for update 5. Add a functionality that extract all the existing branches in 'Flytrex/public_regulated_data_types' repo 6. Add the version to the main window 7. Increment the flytrex version to 0.0.3
There was a problem hiding this comment.
Pull request overview
This PR adds a Flytrex-specific configuration mechanism and UI for selecting which DSDL branch to use for update checks, while also tightening Windows MSI/deploy behavior and surfacing version info in the app title.
Changes:
- Add
config.xmlplus read/write logic and a new “Configurations” menu to select the DSDL branch (and fetch available branches from GitHub). - Improve DSDL packaging/update robustness by validating staged/extracted DSDL content and generating/using a DSDL manifest for pruning stale files.
- Update Windows build/deploy flow (clean stale
cx_Freezeoutputs) and bump Flytrex version to0.0.3.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
winbuild.bat |
Cleans build/ and dist/ before packaging to avoid stale cx_Freeze outputs. |
setup.py |
Stages config.xml, validates bundled DSDL, and writes a DSDL manifest into the staged specs for MSI builds. |
dronecan_gui_tool/version.py |
Increments Flytrex version tuple to 0.0.3. |
dronecan_gui_tool/main.py |
Adds config read/write, Configurations menu, branch discovery worker, selected-branch update checks, DSDL extraction validation, and version in window title. |
config.xml |
Adds default configuration file with the default DSDL branch. |
bin/dronecan_gui_tool |
Adds manifest-based pruning of installed DSDL specs at startup to remove stale files. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
2. Relocate the config.xml file to the user location %APPDATA%
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.