Skip to content

Rename ssdtprojectsystem.targets to SQLProjectsSystem.targets#751

Merged
Phantom-Sixth-Man merged 3 commits intomicrosoft:mainfrom
Phantom-Sixth-Man:kumarshivam/rename-ssdtprojectsystem-to-SQLProjectsSystem
Feb 10, 2026
Merged

Rename ssdtprojectsystem.targets to SQLProjectsSystem.targets#751
Phantom-Sixth-Man merged 3 commits intomicrosoft:mainfrom
Phantom-Sixth-Man:kumarshivam/rename-ssdtprojectsystem-to-SQLProjectsSystem

Conversation

@Phantom-Sixth-Man
Copy link
Contributor

@Phantom-Sixth-Man Phantom-Sixth-Man commented Feb 9, 2026

Description

Adds support for the new SQLProjectsSystem.targets file used by SSMS and other IDEs, while maintaining backward compatibility with the existing ssdtprojectsystem.targets used by SSDT.

Background
The original ssdtprojectsystem.targets was specific to SSDT (SQL Server Data Tools). As SDK-style SQL projects are now supported by additional products like SSMS, a more generic SQLProjectsSystem.targets file is being introduced by those products.

Changes
Updated the import logic in Sdk.targets to:

Import ssdtprojectsystem.targets when it exists (SSDT installed) and SQLProjectsSystem.targets does not exist
Import SQLProjectsSystem.targets when it exists (SSMS/other IDEs installed) and ssdtprojectsystem.targets does not exist
These two targets files are mutually exclusive by design - only one will exist on a machine depending on which product is installed.

Backward Compatibility

  • No breaking changes - existing SSDT installations continue to work as before
  • The SDK automatically detects which targets file is available and uses the appropriate one
  • No action required from the customers.

Code Changes

  • Unit tests are added, if possible - N/A
  • Existing tests are passing
  • New or updated code follows the guidelines here
  • Ensure .dacpac from changes to Microsoft.Build.Sql build process MUST be backwards compatible with older versions of SqlPackage
  • Use proper logging for MSBuild tasks

@Phantom-Sixth-Man Phantom-Sixth-Man marked this pull request as ready for review February 9, 2026 14:17
@Phantom-Sixth-Man Phantom-Sixth-Man marked this pull request as draft February 9, 2026 14:18
@Matteo-T Matteo-T self-requested a review February 10, 2026 12:01
Copy link

@Matteo-T Matteo-T left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Copy link

@Matteo-T Matteo-T left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Phantom-Sixth-Man Phantom-Sixth-Man marked this pull request as ready for review February 10, 2026 12:28
@Phantom-Sixth-Man Phantom-Sixth-Man merged commit f962d63 into microsoft:main Feb 10, 2026
12 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.

3 participants