pynwb
pynwb copied to clipboard
Make ImagingPlane.description optional
Motivation
Fix #2049
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 . && codespellfrom 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.
Codecov Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 94.96%. Comparing base (618df86) to head (03f20ff).
:warning: Report is 18 commits behind head on dev.
Additional details and impacted files
@@ Coverage Diff @@
## dev #2051 +/- ##
==========================================
+ Coverage 94.95% 94.96% +0.01%
==========================================
Files 28 28
Lines 2832 2840 +8
Branches 725 729 +4
==========================================
+ Hits 2689 2697 +8
Misses 87 87
Partials 56 56
| Flag | Coverage Δ | |
|---|---|---|
| integration | 74.33% <0.00%> (-0.21%) |
:arrow_down: |
| unit | 84.82% <100.00%> (+0.04%) |
: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.
Looks good. Thank you @stephprince ! Could you please approve?