Skip to content

refactor: convert directory names from kebab-case to snake_case#959

Draft
gspencergoog wants to merge 1 commit intogoogle:mainfrom
gspencergoog:renaming_dirs
Draft

refactor: convert directory names from kebab-case to snake_case#959
gspencergoog wants to merge 1 commit intogoogle:mainfrom
gspencergoog:renaming_dirs

Conversation

@gspencergoog
Copy link
Collaborator

Description

This change renames all directories containing dashes in the 'main' project to use underscores, standardizing on snake_case for directory names to make the naming scheme consistent.

Key changes:

  • Renamed major directories in 'renderers', 'samples', and 'tools'.
  • Updated all path references in TS, JS, JSON, YAML, and Markdown files.
  • Preserved original kebab-case for file names and runtime component selectors.
  • Performed surgical updates in 'angular.json' and 'package.json' to maintain build integrity.

This change renames all directories containing dashes in the 'main' project to use underscores, standardizing on snake_case for directory names.

Key changes:
- Renamed major directories in 'renderers', 'samples', and 'tools'.
- Updated all path references in TS, JS, JSON, YAML, and Markdown files.
- Preserved original kebab-case for file names and runtime component selectors.
- Fixed accidental refactor of external npm packages and component selectors.
- Performed surgical updates in 'angular.json' and 'package.json' to maintain build integrity.
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request refactors numerous file paths, package names, import statements, and component selectors across the repository to standardize naming conventions by replacing hyphens with underscores. This change affects documentation, renderer configurations, and various client-side sample projects (Angular, Lit, React). No specific feedback is provided in the review comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant