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
Workspaces as windows drives #1159
Conversation
6ac08e7
to
eb41170
Compare
a444b33
to
8d183ce
Compare
Codecov Report
@@ Coverage Diff @@
## master #1159 +/- ##
==========================================
+ Coverage 79.69% 80.06% +0.36%
==========================================
Files 280 281 +1
Lines 23606 23726 +120
==========================================
+ Hits 18813 18996 +183
+ Misses 4793 4730 -63
Continue to review full report at Codecov.
|
84fbf32
to
7b5f726
Compare
902d24d
to
9c66f34
Compare
76ddcbe
to
0a81540
Compare
self.reset_timer = QTimer() | ||
self.reset_timer.setInterval(1000) | ||
self.reset_timer.setInterval(self.RESET_TIMER_THRESHOLD) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't we use signals (fs.entry.updated
I guess) instead of polling ?
If needed, the change would be for another PR of course ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This timer is not used for polling, it is used for throttling the requests to reset the GUI.
assert await bar_txt.exists() | ||
|
||
|
||
@pytest.mark.trio | ||
@pytest.mark.mountpoint | ||
@pytest.mark.skipif(os.name == "nt", reason="Windows uses drive") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a test "drive already used" for windows ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll add one!
Replacing: #1142
Fixing: #928, #1058, #1081, #1082, #925, #913, #914, #912, #1126
Todo: