Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PR: Check for asset availability when checking for update #22598

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

mrclary
Copy link
Contributor

@mrclary mrclary commented Sep 27, 2024

Description of Changes

  • Check that the download asset is available before alerting the user to an available update. If the asset is not available, also check the next-latest release, and so on, until either no update or no asset is available.
  • Use osascript in order not to pollute shell history on macOS.
  • Ensure UpdateWorker.channel is assigned.

Issue(s) Resolved

Fixes #22566
Fixes #22572

@mrclary mrclary changed the title PR: Update manager PR: Check for asset availability when checking for update Sep 27, 2024
…o an available update.

Create a function that determines the asset name, update type, and url, depending on the latest release and the current Spyder version.
Fixes spyder-ide#22566
…WorkerDownloadInstaller.latest_release are now packaging.version.Version objects instead of strings.

We can now use packaging.version mechanisms for comparison, sorting, and stable release checks.
Added test_update_no_asset and test_get_asset_info
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant