Skip to content

Fix nitpicks in id and title Action fields in samples menu#6267

Merged
DragaDoncila merged 2 commits intonapari:mainfrom
lucyleeow:samples
Sep 27, 2023
Merged

Fix nitpicks in id and title Action fields in samples menu#6267
DragaDoncila merged 2 commits intonapari:mainfrom
lucyleeow:samples

Conversation

@lucyleeow
Copy link
Copy Markdown
Contributor

@lucyleeow lucyleeow commented Sep 26, 2023

References and relevant issues

Clean up of missed items from #4865

Description

  • use : instead of . between words in command ID, to make consistent with all other commands which all use : (and fix tests)
  • replace & with && in full menu title, not just the sample display name

@github-actions github-actions bot added tests Something related to our tests qt Relates to qt labels Sep 26, 2023
@lucyleeow
Copy link
Copy Markdown
Contributor Author

cc @DragaDoncila

@lucyleeow lucyleeow changed the title Fix Action in samples menu Fix nitpicks in id and title Action fields in samples menu Sep 26, 2023
@codecov
Copy link
Copy Markdown

codecov bot commented Sep 26, 2023

Codecov Report

Merging #6267 (8cdb5d9) into main (d06b7c0) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #6267      +/-   ##
==========================================
- Coverage   91.60%   91.57%   -0.04%     
==========================================
  Files         585      585              
  Lines       51392    51392              
==========================================
- Hits        47080    47063      -17     
- Misses       4312     4329      +17     
Files Coverage Δ
napari/_qt/_qapp_model/_tests/test_file_menu.py 100.00% <100.00%> (ø)
napari/_qt/dialogs/_tests/test_reader_dialog.py 100.00% <100.00%> (ø)
napari/plugins/_npe2.py 83.71% <100.00%> (ø)

... and 6 files with indirect coverage changes

Copy link
Copy Markdown
Contributor

@DragaDoncila DragaDoncila left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @lucyleeow!

@DragaDoncila DragaDoncila added this to the 0.5.0 milestone Sep 26, 2023
@DragaDoncila DragaDoncila added the ready to merge Last chance for comments! Will be merged in ~24h label Sep 26, 2023
@Czaki Czaki added the maintenance PR with maintance changes, label Sep 26, 2023
@DragaDoncila DragaDoncila merged commit 9efecd7 into napari:main Sep 27, 2023
@lucyleeow lucyleeow deleted the samples branch September 27, 2023 06:45
@Carreau Carreau removed the ready to merge Last chance for comments! Will be merged in ~24h label Sep 27, 2023
kne42 added a commit to kne42/napari that referenced this pull request Sep 27, 2023
* main: (21 commits)
  [pre-commit.ci] pre-commit autoupdate (napari#6266)
  ENH Make `_open_preferences_dialog` return `PreferencesDialog` (napari#6269)
  Fix link to artifacts in update dependecies comment (napari#6270)
  ci(dependabot): bump docker/login-action from 2.1.0 to 3.0.0 (napari#6263)
  FIX add `changing` event to `EventedDict` (napari#6268)
  Restore 'V' keybinding for layer visibiltiy toggle (napari#6261)
  remove: napari.qt.progress (deprecated in 0.4.11) (napari#6252)
  Fix `ShapeList.outline` validations for `int`/list like argument and add a test (napari#6215)
  Fix nitpicks in `id` and `title` `Action` fields in samples menu (napari#6267)
  ci(dependabot): bump actions/checkout from 2 to 4 (napari#6264)
  ci(dependabot): bump actions/setup-python from 2 to 4 (napari#6201)
  Fix typing in _app_model (napari#6059)
  Update `app-model`, `dask`, `fsspec`, `hypothesis`, `imageio`, `ipython`, `jsonschema`, `matplotlib`, `numpy`, `pandas`, `pillow`, `psygnal`, `pytest`, `qtconsole`, `qtpy`, `rich`, `scipy`, `superqt`, `tensorstore`, `tifffile`, `virtualenv`, `xarray`, `zarr` (napari#6265)
  Fix upgrade dependencies wokflow (napari#6211)
  Type _WeakCounter (napari#6246)
  convert Color to string (napari#6243)
  Partially fix translations testing (napari#6014)
  Allow to use all ViewerModel kwargs in Viewer constructor (napari#6209)
  MAINT: Replace `assert np.all(? == ?)` with `assert_array_equal` (napari#6244)
  Fix Python 3.11 StrEnum Compatibility (napari#6242)
  ...
kne42 added a commit to kne42/napari that referenced this pull request Sep 28, 2023
* main: (21 commits)
  [pre-commit.ci] pre-commit autoupdate (napari#6266)
  ENH Make `_open_preferences_dialog` return `PreferencesDialog` (napari#6269)
  Fix link to artifacts in update dependecies comment (napari#6270)
  ci(dependabot): bump docker/login-action from 2.1.0 to 3.0.0 (napari#6263)
  FIX add `changing` event to `EventedDict` (napari#6268)
  Restore 'V' keybinding for layer visibiltiy toggle (napari#6261)
  remove: napari.qt.progress (deprecated in 0.4.11) (napari#6252)
  Fix `ShapeList.outline` validations for `int`/list like argument and add a test (napari#6215)
  Fix nitpicks in `id` and `title` `Action` fields in samples menu (napari#6267)
  ci(dependabot): bump actions/checkout from 2 to 4 (napari#6264)
  ci(dependabot): bump actions/setup-python from 2 to 4 (napari#6201)
  Fix typing in _app_model (napari#6059)
  Update `app-model`, `dask`, `fsspec`, `hypothesis`, `imageio`, `ipython`, `jsonschema`, `matplotlib`, `numpy`, `pandas`, `pillow`, `psygnal`, `pytest`, `qtconsole`, `qtpy`, `rich`, `scipy`, `superqt`, `tensorstore`, `tifffile`, `virtualenv`, `xarray`, `zarr` (napari#6265)
  Fix upgrade dependencies wokflow (napari#6211)
  Type _WeakCounter (napari#6246)
  convert Color to string (napari#6243)
  Partially fix translations testing (napari#6014)
  Allow to use all ViewerModel kwargs in Viewer constructor (napari#6209)
  MAINT: Replace `assert np.all(? == ?)` with `assert_array_equal` (napari#6244)
  Fix Python 3.11 StrEnum Compatibility (napari#6242)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance PR with maintance changes, qt Relates to qt tests Something related to our tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants