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

[FIX] Replace exec_ with exec and fix deprecations #887

Merged
merged 1 commit into from
Aug 22, 2022

Conversation

PrimozGodec
Copy link
Collaborator

Issue
  • In PyQt6, they completely removed exec_ for opening dialogues as of PyQt5 already, exec is available.
  • addSpace keyword is deprecated and, in my opinion, not needed (the user interface already looks good with normal spacing)
Description of changes
  • replacing exec_ with exec.
  • removing addSpace in several widgets. It can be replaced by a separator, but in my opinion, it is not necessary.
Includes
  • Code changes
  • Tests
  • Documentation

@codecov-commenter
Copy link

Codecov Report

Merging #887 (8803c72) into master (40401fe) will increase coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #887      +/-   ##
==========================================
+ Coverage   76.91%   76.92%   +0.01%     
==========================================
  Files          85       85              
  Lines       11984    11984              
  Branches     1883     1883              
==========================================
+ Hits         9217     9219       +2     
+ Misses       2460     2459       -1     
+ Partials      307      306       -1     

@PrimozGodec PrimozGodec merged commit c2dcbdb into biolab:master Aug 22, 2022
@PrimozGodec PrimozGodec deleted the fix-pyqt6 branch August 22, 2022 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants