-
Notifications
You must be signed in to change notification settings - Fork 76
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
UI-searchable API methods/attributes #3384
base: main
Are you sure you want to change the base?
Conversation
Tests won't pass due to deprecated methods until rebased on #3385. In the future, if we have more deprecated plugin methods, we'll either have to manually exclude them from this list or filter the deprecation warning when building the list. |
12623c8
to
66f0914
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3384 +/- ##
==========================================
+ Coverage 88.03% 88.05% +0.01%
==========================================
Files 127 127
Lines 19722 19729 +7
==========================================
+ Hits 17363 17372 +9
+ Misses 2359 2357 -2 ☔ View full report in Codecov by Sentry. |
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.
Demo was cool. Thanks!
Description
This pull request implements plugin-tray filtering of plugins by their exposed methods/attributes. (Note that this also helps expose to us places where we expose args/kwargs that probably shouldn't be exposed and should work to clean those up 🐱)
Screen.Recording.2025-01-09.at.11.35.59.AM.mov
Note that currently this filtering still happens when API hints are off (so "Plot Options" still appears for a search of "RGB"), but that could easily be changed if preferred.
Change log entry
CHANGES.rst
? If you want to avoid merge conflicts,list the proposed change log here for review and add to
CHANGES.rst
before merge. If no, maintainershould add a
no-changelog-entry-needed
label.Checklist for package maintainer(s)
This checklist is meant to remind the package maintainer(s) who will review this pull request of some common things to look for. This list is not exhaustive.
trivial
label.