search

nucleic / enaml

Declarative User Interfaces for Python
http://enaml.readthedocs.io/en/latest/
Other
1.53k stars 130 forks source link

Ensure some more QPoint values are ints #481

Closed frmdstryr closed 2 years ago

MatthieuDartiailh commented 2 years ago

Looks good. Did you get a chance to audit other QPoint uses across the project ? If not I will try to do it. I would like for this PR to be the last regarding this issue.

frmdstryr commented 2 years ago

After getting an error from the first change I did a search for QPoint and these are the only ones I found. It looks like most were done in #471

MatthieuDartiailh commented 2 years ago

Thanks for checking. Could add a release note entry (maybe just update the line of #471) ? I will merge ASAP.

MatthieuDartiailh commented 2 years ago

Thanks for checking. Could add a release note entry (maybe just update the line of #471) ? I will merge ASAP.