Skip to content

Organize SDK projects in solution - #5721

Open
Brendan Kowitz (brendankowitz) wants to merge 1 commit into
mainfrom
brendankowitz-fix-sdk-solution-folders
Open

Organize SDK projects in solution#5721
Brendan Kowitz (brendankowitz) wants to merge 1 commit into
mainfrom
brendankowitz-fix-sdk-solution-folders

Conversation

@brendankowitz

@brendankowitz Brendan Kowitz (brendankowitz) commented Aug 11, 2026

Copy link
Copy Markdown
Member

Description

Organizes the four Firely SDK projects and the Ignixa provider project beneath a src > SDK solution folder. This improves solution navigation only; project definitions, GUIDs, build configuration, packages, and runtime behavior are unchanged.

image

Related issues

Addresses AB#198706.

Testing

  • dotnet sln Microsoft.Health.Fhir.sln list
  • git diff --check origin/main...HEAD

FHIR Team Checklist

  • PR title is succinct and under 65 characters.
  • Tagged as Enhancement and Open source.
  • No SQL scripts were added or changed.
  • No ADR is required because this does not change system behavior or design assumptions.
  • CI is green before merge Build Status
  • Reviewed squash-merge requirements.

Semver Change (docs)

Skip — solution-folder organization only; no shipped behavior or package changes.

Group Firely SDK and Ignixa provider projects under src/SDK for easier discovery without changing project definitions or build configuration.

Changes:
- Microsoft.Health.Fhir.sln: Add the SDK solution folder and reparent five SDK-related projects.

Tools: skill: codeblend-commit

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@brendankowitz Brendan Kowitz (brendankowitz) added Enhancement Enhancement on existing functionality. Open source This change is only relevant to the OSS code or release. No-PaaS-breaking-change No-ADR ADR not needed labels Aug 11, 2026
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@c7158f4). Learn more about missing BASE report.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #5721   +/-   ##
=======================================
  Coverage        ?   77.79%           
=======================================
  Files           ?     1011           
  Lines           ?    37262           
  Branches        ?     5680           
=======================================
  Hits            ?    28989           
  Misses          ?     6897           
  Partials        ?     1376           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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

Labels

Enhancement Enhancement on existing functionality. No-ADR ADR not needed No-PaaS-breaking-change Open source This change is only relevant to the OSS code or release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants