Skip to content

Conversation

@bendichter
Copy link
Contributor

Add two new inspector checks for related_publications metadata:

  • check_publication_list_format: detects comma-separated DOIs/URLs that should be separate list entries
  • check_publication_doi_resolves: verifies DOI URLs resolve to valid publications via network requests

Includes documentation updates for best practices guidance.

Closes #419

Motivation

What was the reasoning behind this change? Please explain the changes briefly.

bendichter and others added 2 commits December 24, 2025 09:34
Add two new inspector checks for related_publications metadata:
- check_publication_list_format: detects comma-separated DOIs/URLs
  that should be separate list entries
- check_publication_doi_resolves: verifies DOI URLs resolve to valid
  publications via network requests

Includes documentation updates for best practices guidance.

Closes #419
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 83.63636% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.06%. Comparing base (59ef74b) to head (452621c).

Files with missing lines Patch % Lines
src/nwbinspector/checks/_nwbfile_metadata.py 83.63% 9 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #651      +/-   ##
==========================================
+ Coverage   77.41%   81.06%   +3.64%     
==========================================
  Files          47       47              
  Lines        1656     1711      +55     
==========================================
+ Hits         1282     1387     +105     
+ Misses        374      324      -50     
Files with missing lines Coverage Δ
src/nwbinspector/checks/__init__.py 100.00% <ø> (ø)
src/nwbinspector/checks/_nwbfile_metadata.py 93.04% <83.63%> (-2.96%) ⬇️

... and 5 files with indirect coverage changes

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Add Check]: Comma-separated entries in releated_publication should be list of entries

3 participants