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.

Sign up for GitHub

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

Jump to bottom

PR: Add missing QMessageBox import #4762

Merged
merged 1 commit into from Jul 18, 2017
Merged

PR: Add missing QMessageBox import #4762

merged 1 commit into from Jul 18, 2017

Conversation

rlaverde
Copy link
Member

A missing import after the 3.x changes was merged in master

    def show_compatibility_message(self, message):
        """Show compatibility message."""
>       messageBox = QMessageBox(self)
E       NameError: name 'QMessageBox' is not defined

@rlaverde rlaverde self-assigned this Jul 18, 2017
@rlaverde rlaverde requested a review from ccordoba12 July 18, 2017 16:20
@ccordoba12 ccordoba12 added this to the v4.0beta1 milestone Jul 18, 2017
@ccordoba12 ccordoba12 changed the title PR: Add missing import QMessageBox. PR: Add missing QMessageBox import Jul 18, 2017
@ccordoba12 ccordoba12 merged commit 0608076 into spyder-ide:master Jul 18, 2017
@rlaverde rlaverde deleted the fix-missing-import-plugins-api branch September 18, 2017 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ccordoba12 ccordoba12 ccordoba12 approved these changes

Assignees

@rlaverde rlaverde

Labels
None yet
Projects
None yet
Milestone
v4.0beta1
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@rlaverde @ccordoba12