Daniel Althviz Moré

Results 548 comments of Daniel Althviz Moré

Thank you for the extra info @hprodh ! I think currently we don't support `flake8` only `pyflakes`. Checking around I think there is actually an issue to add support for...

Hi there, I think the `napari-console` PR should be merged first. In fact I think a release with the change will be needed so this PR can update the `napari-console`...

I added some changes to the console PR and checked locally with napari 0.4.19 so things seem ready for review now :+1:

> Are we waiting on console to release before this PR can be updated? Correct, in fact https://github.com/napari/napari-console/pull/33 needs to be merged first and then a release with that change...

> WDYT @dalthviz ? If you're okay with this, I think you don't have to keep this PR up to date, as we won't merge it, and instead tackle keybinding...

Hi @wakonig thank you for reaching out! I would say that adding PySide 6.8 should be worthy indeed

Hi @th3w1zard1 and sorry for such a late response! Regarding your initial compatibilty issue related with `QUndoStack` I would say adding over QtPy the missing import at https://github.com/spyder-ide/qtpy/blob/0f7b181e45186b568551cf50078e5eeda05e3953/qtpy/QtWidgets.py#L37-L44 https://github.com/spyder-ide/qtpy/blob/0f7b181e45186b568551cf50078e5eeda05e3953/qtpy/QtWidgets.py#L113-L114 https://github.com/spyder-ide/qtpy/blob/0f7b181e45186b568551cf50078e5eeda05e3953/qtpy/QtGui.py#L54-L64...

> Hey @th3w1zard1, thanks for reporting. Your suggestion sounds reasonable to me. > >@dalthviz, what do you think? The initial suggestion makes sense to me :+1: Also, just in case,...