Skip to content

Renamed files to kebab-case - core - services - part 2#20

Open
tomerqodo wants to merge 2 commits intocopilot_combined_20260121_qodo_grep_cursor_copilot_1_base_renamed_files_to_kebab-case_-_core_-_services__-_part_2_pr224from
copilot_combined_20260121_qodo_grep_cursor_copilot_1_head_renamed_files_to_kebab-case_-_core_-_services__-_part_2_pr224
Open

Renamed files to kebab-case - core - services - part 2#20
tomerqodo wants to merge 2 commits intocopilot_combined_20260121_qodo_grep_cursor_copilot_1_base_renamed_files_to_kebab-case_-_core_-_services__-_part_2_pr224from
copilot_combined_20260121_qodo_grep_cursor_copilot_1_head_renamed_files_to_kebab-case_-_core_-_services__-_part_2_pr224

Conversation

@tomerqodo
Copy link

Benchmark PR from qodo-benchmark#224

Copilot AI review requested due to automatic review settings January 21, 2026 17:55
Copy link

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 pull request renames service files from PascalCase to kebab-case as part 2 of a larger refactoring effort. The changes update require/import statements across test files and service implementations to use the new kebab-case naming convention.

Changes:

  • Updated 70+ require/import statements to reference kebab-case file names
  • Created new service files with kebab-case names (webhook-trigger.js, verification-trigger.js, etc.)
  • Removed trailing whitespace from multiple test files

Reviewed changes

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

File Description
Test files (*.test.js, *.test.ts) Updated require statements to use kebab-case service file names
Service files (*.js, *.ts) Updated internal require/import statements and created new files with kebab-case names
Index files Updated module.exports to reference kebab-case files
Type annotations Updated TypeScript import paths and JSDoc type references

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

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