Skip to content

Conversation

svozza
Copy link
Contributor

@svozza svozza commented Sep 9, 2025

Summary

Renames the BaseRouter class to Router to bring it inline with the naming convention of the other event handlers.

Changes

  • Renames the BaseRouter.ts file to Router.ts
  • Renames the exported class BaseRouter to Router
  • Updates all the tests to use the new class name

Issue number: closes #4447


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

@svozza svozza added this to the Event Handler Beta (priority) milestone Sep 9, 2025
@svozza svozza requested review from dreamorosi and sdangol September 9, 2025 08:45
@svozza svozza self-assigned this Sep 9, 2025
@boring-cyborg boring-cyborg bot added event-handler This item relates to the Event Handler Utility tests PRs that add or change tests labels Sep 9, 2025
@pull-request-size pull-request-size bot added the size/L PRs between 100-499 LOC label Sep 9, 2025
Copy link

sonarqubecloud bot commented Sep 9, 2025

@svozza svozza merged commit b043c28 into main Sep 9, 2025
45 checks passed
@svozza svozza deleted the event-handler/rename-baserouter-class branch September 9, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

event-handler This item relates to the Event Handler Utility size/L PRs between 100-499 LOC tests PRs that add or change tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Maintenance: Rename BaseRouter class in REST handler to Router

2 participants