Skip to content

refactor(igc-ts): centralize Web Components package version constants#1752

Merged
kdinev merged 2 commits into
masterfrom
copilot/update-wc-react
Jul 14, 2026
Merged

refactor(igc-ts): centralize Web Components package version constants#1752
kdinev merged 2 commits into
masterfrom
copilot/update-wc-react

Conversation

Copilot AI commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

This follow-up on update-wc-react applies the same version-centralization pattern used in React templates to Web Components igc-ts templates, so shared package updates are defined once and reused consistently. It removes repeated igniteui-webcomponents-* version strings from component template classes while preserving where project-level dependency declarations belong.

  • What changed

    • Added /packages/cli/templates/webcomponents/igc-ts/constants.ts with shared package constants for:
      • core, charts, gauges, grids, inputs, layouts
    • Refactored igc-ts template index.ts files (including tree/default/index.ts, grid variants, chart/gauge templates, and related grid templates) to import constants instead of hardcoded @~x.y.z strings.
  • Scope boundaries

    • Kept direct dependency declarations in project package.json templates under projects/_base* unchanged, matching existing repository conventions.
    • No template behavior changes intended; this is a source-level deduplication/refactor.
  • Pattern example

    import { IGNITEUI_WEBCOMPONENTS_GRIDS_PACKAGE } from "../../constants";
    
    this.packages = [IGNITEUI_WEBCOMPONENTS_GRIDS_PACKAGE];

Copilot AI changed the title [WIP] Refactor webcomponents templates to centralize package versions refactor(igc-ts): centralize Web Components package version constants Jul 14, 2026
Copilot AI requested a review from damyanpetev July 14, 2026 15:43
Base automatically changed from update-wc-react to master July 14, 2026 15:52
kdinev
kdinev previously approved these changes Jul 14, 2026
@damyanpetev damyanpetev force-pushed the copilot/update-wc-react branch from 627955c to 7ac9226 Compare July 14, 2026 16:20
@damyanpetev damyanpetev marked this pull request as ready for review July 14, 2026 16:23
Copilot AI review requested due to automatic review settings July 14, 2026 16:23
@coveralls

coveralls commented Jul 14, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 87.816% (+0.03%) from 87.79% — copilot/update-wc-react into master

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

Centralizes igniteui-webcomponents-* package version strings for the webcomponents/igc-ts templates into a single constants module, reducing duplication across template definitions and making version updates easier to apply consistently.

Changes:

  • Added a new constants.ts module exporting shared igniteui-webcomponents-* package specifiers (name + version).
  • Updated multiple igc-ts component template index.ts files to import and use these constants instead of hardcoded strings.
  • Updated the igniteui-webcomponents-grids dependency version in the _base_with_home project template package.json.

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/cli/templates/webcomponents/igc-ts/constants.ts Introduces shared constants for Web Components package specifiers.
packages/cli/templates/webcomponents/igc-ts/tree/default/index.ts Replaces hardcoded package strings with imported constants for tree template deps.
packages/cli/templates/webcomponents/igc-ts/tree-grid/basic/index.ts Uses centralized grids package constant.
packages/cli/templates/webcomponents/igc-ts/radial-gauge/default/index.ts Uses centralized core + gauges package constants.
packages/cli/templates/webcomponents/igc-ts/linear-gauge/default/index.ts Uses centralized core + gauges package constants.
packages/cli/templates/webcomponents/igc-ts/pie-chart/default/index.ts Uses centralized core + charts package constants.
packages/cli/templates/webcomponents/igc-ts/financial-chart/default/index.ts Uses centralized core + charts package constants.
packages/cli/templates/webcomponents/igc-ts/grid/default/index.ts Uses centralized grids package constant.
packages/cli/templates/webcomponents/igc-ts/grid/grid-editing/index.ts Uses centralized grids package constant.
packages/cli/templates/webcomponents/igc-ts/grid/grid-summaries/index.ts Uses centralized grids package constant.
packages/cli/templates/webcomponents/igc-ts/hierarchical-grid/basic/index.ts Uses centralized grids package constant.
packages/cli/templates/webcomponents/igc-ts/pivot-grid/basic/index.ts Uses centralized grids package constant.
packages/cli/templates/webcomponents/igc-ts/projects/_base_with_home/files/package.json Bumps igniteui-webcomponents-grids version to ~7.2.1.

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

"igniteui-webcomponents-core": "~7.0.0",
"igniteui-webcomponents-gauges": "~7.0.0",
"igniteui-webcomponents-grids": "~7.1.0",
"igniteui-webcomponents-grids": "~7.2.1",
@kdinev kdinev merged commit d290548 into master Jul 14, 2026
5 checks passed
@kdinev kdinev deleted the copilot/update-wc-react branch July 14, 2026 16:27
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.

5 participants