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: Add validation to prevent calling uninitialized application_update_status widget on conda installations (Application) #20036

Merged
merged 1 commit into from
Nov 12, 2022

Conversation

dalthviz
Copy link
Member

Description of Changes

The application update status bar is only initialized for the standalone installers but no validations were added to prevent calling it inside the check updates logic available for all types of Spyder installations (standalone installers, conda, pip, etc).

Issue(s) Resolved

Fixes #20035

I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: dalthviz

…ion_update_status widget on conda installations
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks @dalthviz!

@ccordoba12 ccordoba12 changed the title PR: Add validation to prevent calling uninitialized application_update_status widget on conda installations (Application) PR: Add validation to prevent calling uninitialized application_update_status widget on conda installations (Application) Nov 12, 2022
@ccordoba12 ccordoba12 changed the title PR: Add validation to prevent calling uninitialized application_update_status widget on conda installations (Application) PR: Add validation to prevent calling uninitialized application_update_status widget on conda installations (Application) Nov 12, 2022
@ccordoba12 ccordoba12 merged commit aac0adb into spyder-ide:5.x Nov 12, 2022
ccordoba12 added a commit that referenced this pull request Nov 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants