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 a dialog to report internal errors #382

Merged
merged 10 commits into from
Apr 20, 2021

Conversation

jnsebgosselin
Copy link
Owner

I implemented this as part of another project, so it was not a big job to port the work here and I think it will be quite useful. Ultimately, I want to get rid of the cmd window.

image

@pep8speaks
Copy link

Hello @jnsebgosselin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 28:1: E402 module level import not at top of file
Line 29:1: E402 module level import not at top of file
Line 34:1: E402 module level import not at top of file
Line 35:1: E402 module level import not at top of file

@jnsebgosselin jnsebgosselin merged commit c15326b into master Apr 20, 2021
@jnsebgosselin jnsebgosselin deleted the add_internal_error_dialog branch April 20, 2021 21:08
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