search

napari / docs

Documentation for napari (other than API docs)
BSD 3-Clause "New" or "Revised" License
9 stars 37 forks source link

Update where app-model actions/providers/processors live in codebase #416

Closed lucyleeow closed 4 weeks ago

lucyleeow commented 2 months ago

References and relevant issues

Relates to https://github.com/napari/napari/pull/6848 Depends on https://github.com/napari/napari/pull/6743

Description

Updates the info on where actions live in the napari codebase.

lucyleeow commented 2 months ago

cc @DragaDoncila

lucyleeow commented 2 months ago

Thanks, changes made!

lucyleeow commented 2 months ago

AH whoops sorry, I forgot this has already been approved.

I realise that we now no longer have qt providers or processors anymore, so I've updated that here too. Sorry!

DragaDoncila commented 2 months ago

@lucyleeow do we now have to wait for #6743 to go in before we merge this?

lucyleeow commented 2 months ago

:facepalm: yes you are right. I'll add a depends on

Czaki commented 2 months ago

test comment

lucyleeow commented 1 month ago

The depends on has now been merged (not sure why the CI is still failing) so maybe this could go in now, @DragaDoncila ?

DragaDoncila commented 4 weeks ago

thanks @lucyleeow sorry for the delay...