Skip to content

refactor: integrate extended TOML file into working config#15848

Merged
christopherco merged 1 commit intomicrosoft:tomls/base/mainfrom
reubeno:adjust
Feb 14, 2026
Merged

refactor: integrate extended TOML file into working config#15848
christopherco merged 1 commit intomicrosoft:tomls/base/mainfrom
reubeno:adjust

Conversation

@reubeno
Copy link
Member

@reubeno reubeno commented Feb 14, 2026

  • Ensures that the "full" component list is included in the processed TOML config tree
  • Removes duplicate component names.

@reubeno reubeno marked this pull request as ready for review February 14, 2026 03:25
Copilot AI review requested due to automatic review settings February 14, 2026 03:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR refactors the component configuration structure to properly integrate components-full.toml into the main component configuration tree while removing duplicate entries.

Changes:

  • Explicitly includes components-full.toml in the includes array of base/comps/components.toml
  • Removes redundant includes directive from components-full.toml to prevent circular/duplicate loading
  • Removes ~300 duplicate component declarations from components-full.toml that already exist in components.toml or individual .comp.toml files

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
base/comps/components.toml Adds explicit reference to components-full.toml in includes array to ensure full component list is loaded
base/comps/components-full.toml Removes redundant includes directive, updates description to clarify purpose (build-time dependency closure), and removes ~300 duplicate component entries that exist elsewhere

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@christopherco christopherco merged commit 443dabd into microsoft:tomls/base/main Feb 14, 2026
12 checks passed
@reubeno reubeno deleted the adjust branch February 14, 2026 03:31
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.

2 participants