Skip to content

fix(mcp): add missing submodules to switch-submodules script - #1781

Merged
dkalinovInfra merged 1 commit into
masterfrom
dkalinov/mcp-doc-pipeline-fix
Aug 28, 2026
Merged

fix(mcp): add missing submodules to switch-submodules script#1781
dkalinovInfra merged 1 commit into
masterfrom
dkalinov/mcp-doc-pipeline-fix

Conversation

@dkalinovInfra

Copy link
Copy Markdown
Contributor

Description

Related Issue

Closes #

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring / code cleanup
  • Build / CI configuration change

Affected Packages

  • igniteui-cli (packages/cli)
  • @igniteui/cli-core (packages/core)
  • @igniteui/angular-templates (packages/igx-templates)
  • @igniteui/angular-schematics (packages/ng-schematics)
  • @igniteui/mcp-server (packages/igniteui-mcp)

Checklist

  • I have tested my changes locally (npm run test)
  • I have built the project successfully (npm run build)
  • I have run the linter (npm run lint)
  • I have added/updated tests as needed
  • My changes do not introduce new warnings or errors

Additional Context

Copilot AI lite review requested due to automatic review settings August 28, 2026 06:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the MCP documentation workspace’s switch-submodules.sh helper so it also switches the Angular doc pipeline submodules that are initialized in CI/local runs, preventing them from being left on the default pinned commit when a non-default branch is requested.

Changes:

  • Add angular/igniteui-angular-samples to the list of submodules to switch.
  • Add angular/igniteui-docfx to the list of submodules to switch.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 87.816%. remained the same — dkalinov/mcp-doc-pipeline-fix into master

@dkalinovInfra
dkalinovInfra enabled auto-merge (squash) August 28, 2026 06:23
@dkalinovInfra
dkalinovInfra disabled auto-merge August 28, 2026 06:24
@dkalinovInfra
dkalinovInfra merged commit 064db42 into master Aug 28, 2026
5 checks passed
@dkalinovInfra
dkalinovInfra deleted the dkalinov/mcp-doc-pipeline-fix branch August 28, 2026 06:24
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