Brainrender napari > Brainrender

alessandrofelder commented 10 months ago

Description

What is this PR

[x] Bug fix
[ ] Addition of a new feature
[ ] Other

Why is this PR needed? It is unnecessary to specify that the widget is a napari widget if the widget is inside napari anyway.

What does this PR do? Changes the display names of the brainrender widget.

References

Closes #71

How has this PR been tested?

Manual testing and looking at the GUI.

Is this a breaking change?

Nope

Does this PR require an update to the documentation?

Nope

Checklist:

[x] The code has been tested locally
[n/a] Tests have been added to cover all new functionality (unit & integration)
[n/a] The documentation has been updated to reflect any changes
[x] The code has been formatted with pre-commit

codecov[bot] commented 10 months ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (7121f2d) 96.72% compared to head (9a61363) 96.72%.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #73 +/- ## ======================================= Coverage 96.72% 96.72% ======================================= Files 7 7 Lines 305 305 ======================================= Hits 295 295 Misses 10 10 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

alessandrofelder commented 10 months ago

Windows tests were taking too long here and elsewhere, so opened https://github.com/brainglobe/brainrender-napari/issues/74

brainglobe / brainrender-napari