Skip to content

Commit 20655d2

Browse files
Ebram TawfikCopilot
andcommitted
Add DataFactory.MCP.Core as submodule reference in Fabric.Mcp.Server
- Add DataFactory.MCP as git submodule at external/DataFactory.MCP - Add project reference using $(RepoRoot) for portability - Add Apache.Arrow 22.1.0 to Directory.Packages.props (only new dependency needed) - Microsoft.Identity.Client and Microsoft.Extensions.Http already satisfied transitively - Build verified: 0 warnings, 0 errors Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 2ad0e00 commit 20655d2

5 files changed

Lines changed: 18 additions & 5 deletions

File tree

‎.gitmodules‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
[submodule "external/DataFactory.MCP"]
2+
path = external/DataFactory.MCP
3+
url = https://github.com/microsoft/DataFactory.MCP.git
4+
branch = fabric-mcp-integration

‎CONTRIBUTING.md‎

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,9 @@ If you are contributing significant changes, or if the issue is already assigned
9292
- `{server}.Mcp.Tools.{tool-name}.UnitTests/` - Unit tests require no authentication or test resources
9393
- `{server}.Mcp.Tools.{tool-name}.LiveTests/` - Live tests depend on Azure resources and authentication
9494
- `test-resources.bicep` - Infrastructure templates for testing
95+
- `external/` - Git submodules for external project integrations
96+
- `DataFactory.MCP` - [DataFactory.MCP](https://github.com/microsoft/DataFactory.MCP) submodule (used by `Fabric.Mcp.Server`)
97+
- Run `git submodule update --init --recursive` after cloning to fetch submodule contents
9598
- `eng/` - Shared tools, templates, CLI helpers
9699
- `docs/` - Central documentation and onboarding materials
97100

@@ -100,11 +103,13 @@ If you are contributing significant changes, or if the issue is already assigned
100103
### Development Process
101104

102105
1. Fork the repository
103-
2. Create a feature branch
104-
3. Make your changes
105-
4. Write or update tests
106-
5. Test locally
107-
6. Submit a pull request
106+
2. Clone with submodules: `git clone --recurse-submodules <your-fork-url>`
107+
- If already cloned, run: `git submodule update --init --recursive`
108+
3. Create a feature branch
109+
4. Make your changes
110+
5. Write or update tests
111+
6. Test locally
112+
7. Submit a pull request
108113

109114
### Adding a New Command
110115

‎Directory.Packages.props‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
<ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
44
</PropertyGroup>
55
<ItemGroup>
6+
<PackageVersion Include="Apache.Arrow" Version="22.1.0" />
67
<PackageVersion Include="Azure.Containers.ContainerRegistry" Version="1.2.0" />
78
<PackageVersion Include="Azure.Communication.Sms" Version="1.0.2" />
89
<PackageVersion Include="Azure.Communication.Email" Version="1.0.2" />

‎external/DataFactory.MCP‎

Submodule DataFactory.MCP added at 950dc5c

‎servers/Fabric.Mcp.Server/src/Fabric.Mcp.Server.csproj‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,8 @@
6969
<ItemGroup>
7070
<ProjectReference Include="$(RepoRoot)\core\Fabric.Mcp.Core\src\Fabric.Mcp.Core.csproj" />
7171
<ProjectReference Include="$(RepoRoot)\tools\Fabric.*\src\*.csproj" />
72+
<!-- DataFactory.MCP.Core integration (via git submodule at external/DataFactory.MCP) -->
73+
<ProjectReference Include="$(RepoRoot)\external\DataFactory.MCP\DataFactory.MCP.Core\DataFactory.MCP.Core.csproj" />
7274
</ItemGroup>
7375

7476
<ItemGroup>

0 commit comments

Comments
 (0)