Skip to content

Conversation

@chennes
Copy link
Member

@chennes chennes commented Aug 18, 2025

No description provided.

@chennes chennes requested a review from Copilot August 18, 2025 00:29

This comment was marked as outdated.

@chennes chennes force-pushed the devAddCheckForMissingDepsOnStartup branch from 04fd74a to 8b055dc Compare August 18, 2025 00:51
@chennes chennes requested a review from Copilot August 18, 2025 00:51
Copy link
Contributor

Copilot AI left a comment

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 functionality to check for missing dependencies on startup in the Addon Manager. The system now automatically scans installed addons for missing workbenches, external addons, and Python packages, then presents an optional dialog to help users install them.

  • Implements a new CheckForMissingDependenciesWorker to scan for missing dependencies during startup
  • Adds UI dialog functionality to handle missing dependency installation
  • Introduces a preference system to allow users to ignore specific missing dependencies

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
addonmanager_workers_startup.py Adds new worker class to check for missing dependencies across installed addons
addonmanager_preferences_defaults.json Adds preference for tracking ignored missing dependencies
addonmanager_installer_gui.py Moves dependency dialog initialization to constructor for reuse
AddonManager.py Integrates missing dependency checking into startup sequence and adds post-startup dialog handling
Addon.py Adds join method to combine multiple MissingDependencies objects

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@chennes chennes merged commit b919989 into FreeCAD:dev Aug 18, 2025
1 check passed
@chennes chennes deleted the devAddCheckForMissingDepsOnStartup branch August 18, 2025 00:55
@chennes chennes added the release to main Trigger an Action to create a PR backporting to `main` label Aug 24, 2025
@github-actions
Copy link

Successfully created backport PR for main:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release to main Trigger an Action to create a PR backporting to `main`

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant