pynwb icon indicating copy to clipboard operation
pynwb copied to clipboard

Deprecate NWBGroupSpec.add_group, add_dataset

Open rly opened this issue 2 months ago • 1 comments

Motivation

Coordinating with https://github.com/hdmf-dev/hdmf/pull/1333

How to test the behavior?

python test.py

Checklist

  • [x] Did you update CHANGELOG.md with your changes?
  • [x] Have you checked our Contributing document?
  • [x] Have you ensured the PR clearly describes the problem and the solution?
  • [x] Is your contribution compliant with our coding style? This can be checked running ruff check . && codespell from the source directory.
  • [x] Have you checked to ensure that there aren't other open Pull Requests for the same change?
  • [x] Have you included the relevant issue number using "Fix #XXX" notation where XXX is the issue number? By including "Fix #XXX" you allow GitHub to close issue #XXX when the PR is merged.

rly avatar Oct 08 '25 20:10 rly

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 95.24%. Comparing base (4c28a99) to head (150fad4).

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #2138   +/-   ##
=======================================
  Coverage   95.23%   95.24%           
=======================================
  Files          29       29           
  Lines        2876     2879    +3     
  Branches      740      740           
=======================================
+ Hits         2739     2742    +3     
  Misses         83       83           
  Partials       54       54           
Flag Coverage Δ
integration 74.40% <33.33%> (-0.05%) :arrow_down:
unit 84.96% <100.00%> (+0.01%) :arrow_up:

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

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

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

codecov[bot] avatar Oct 08 '25 20:10 codecov[bot]