Skip to content

fix: Refactor DICOM Tag Browser path check in Header#369

Merged
igoroctaviano merged 1 commit intomasterfrom
fix/dicom-tag-browser-rendering
Feb 4, 2026
Merged

fix: Refactor DICOM Tag Browser path check in Header#369
igoroctaviano merged 1 commit intomasterfrom
fix/dicom-tag-browser-rendering

Conversation

@igoroctaviano
Copy link
Collaborator

  • Replace repeated pathname.includes() calls with a single .some() over a shared path list
  • Add module-level DICOM_TAG_BROWSER_PATHS constant for easier maintenance
  • Avoid recreating the path array on each render

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 4, 2026

@deepsource-io
Copy link

deepsource-io bot commented Feb 4, 2026

Here's the code health analysis summary for commits bcf57c9..b544192. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript✅ SuccessView Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

@igoroctaviano igoroctaviano merged commit 894f63f into master Feb 4, 2026
8 checks passed
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.

1 participant