Skip to content

Bump nwbinspector to ~= 0.7.0#1809

Open
bendichter wants to merge 3 commits intomasterfrom
bump-nwbinspector-0.7.0
Open

Bump nwbinspector to ~= 0.7.0#1809
bendichter wants to merge 3 commits intomasterfrom
bump-nwbinspector-0.7.0

Conversation

@bendichter
Copy link
Member

Summary

  • Bumps nwbinspector dependency from >= 0.4.28,!=0.4.32 to ~= 0.7.0

Test plan

  • CI passes with the new nwbinspector version

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Feb 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.11%. Comparing base (a0315c1) to head (0282f54).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1809      +/-   ##
==========================================
- Coverage   75.12%   75.11%   -0.01%     
==========================================
  Files          84       84              
  Lines       11925    11925              
==========================================
- Hits         8959     8958       -1     
- Misses       2966     2967       +1     
Flag Coverage Δ
unittests 75.11% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yarikoptic yarikoptic added the patch Increment the patch version when merged label Feb 24, 2026
@yarikoptic
Copy link
Member

dandi/tests/test_dandiarchive.py::test_follow_redirect stalled on windows, I restarted but it is unrelated, so let's just proceed
dandi/tests/test_dandiarchive.py::test_follow_redirect +++++++++++++++++++++++++++++++++++ Timeout +++++++++++++++++++++++++++++++++++
~~~~~~~~~~~~~~~~~~~~~~ Stack of QueueFeederThread (9100) ~~~~~~~~~~~~~~~~~~~~~~
  File "C:\hostedtoolcache\windows\Python\3.10.11\x64\lib\threading.py", line 973, in _bootstrap
    self._bootstrap_inner()
  File "C:\hostedtoolcache\windows\Python\3.10.11\x64\lib\threading.py", line 1016, in _bootstrap_inner
    self.run()
  File "C:\hostedtoolcache\windows\Python\3.10.11\x64\lib\threading.py", line 953, in run
    self._target(*self._args, **self._kwargs)
  File "C:\hostedtoolcache\windows\Python\3.10.11\x64\lib\site-packages\joblib\externals\loky\backend\queues.py", line 147, in _feed
    nwait()
  File "C:\hostedtoolcache\windows\Python\3.10.11\x64\lib\threading.py", line 320, in wait
    waiter.acquire()
~~~~~~~~~~~~~~~~~~~~ Stack of ExecutorManagerThread (1732) ~~~~~~~~~~~~~~~~~~~~
  File "C:\hostedtoolcache\windows\Python\3.10.11\x64\lib\threading.py", line 973, in _bootstrap

@yarikoptic
Copy link
Member

on the other hand -- why we need to depend/restrict to 0.7 series of nwbinspector, are you planing to break in 0.8 series?

note that we tend to maintain minimal compatible versions, and not to restrict top versions unless necessary.

@bendichter what was actual motivation now, as why not before?

@bendichter
Copy link
Member Author

No particular reason. Feel free to keep it open ended

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

patch Increment the patch version when merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants