Skip to content

feature/AddConfigMenu#9

Merged
ilang-flytrex merged 2 commits into
masterfrom
feature/AddConfigMenu
May 24, 2026
Merged

feature/AddConfigMenu#9
ilang-flytrex merged 2 commits into
masterfrom
feature/AddConfigMenu

Conversation

@ilang-flytrex

Copy link
Copy Markdown
  1. Add config.xml configuration file
  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

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
@ilang-flytrex ilang-flytrex requested a review from Copilot May 20, 2026 10:31
@ilang-flytrex ilang-flytrex self-assigned this May 20, 2026
@ilang-flytrex ilang-flytrex added the enhancement New feature or request label May 20, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.xml plus 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_Freeze outputs) and bump Flytrex version to 0.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.

Comment thread dronecan_gui_tool/main.py Outdated
Comment thread dronecan_gui_tool/main.py Outdated
Comment thread setup.py Outdated
Comment thread config.xml Outdated
2. Relocate the config.xml file to the user location %APPDATA%
@ilang-flytrex ilang-flytrex merged commit 4ea6ba6 into master May 24, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants