Skip to content

Conversation

@Gudahtt
Copy link
Member

@Gudahtt Gudahtt commented Aug 19, 2025

Explanation

Remove the isBaseController utility function.

References

Fixes #6340

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@Gudahtt Gudahtt force-pushed the remove-is-base-controller branch from d1af3c4 to 7c0ecbf Compare August 19, 2025 20:56
@Gudahtt Gudahtt force-pushed the messenger-migrate-base-controller branch 2 times, most recently from ae71891 to 67c7853 Compare August 20, 2025 14:04
Base automatically changed from messenger-migrate-base-controller to main August 20, 2025 14:10
@Gudahtt Gudahtt force-pushed the remove-is-base-controller branch from 7c0ecbf to 8163913 Compare August 20, 2025 14:13
@Gudahtt Gudahtt marked this pull request as ready for review August 20, 2025 14:20
@Gudahtt Gudahtt requested a review from a team as a code owner August 20, 2025 14:20
@Gudahtt Gudahtt marked this pull request as draft August 20, 2025 14:20
@Gudahtt

This comment was marked as resolved.

Remove the `isBaseController` utility function.

Fixes #6340
@Gudahtt Gudahtt force-pushed the remove-is-base-controller branch from 8163913 to 5fddbe3 Compare August 20, 2025 16:14
@Gudahtt Gudahtt marked this pull request as ready for review August 20, 2025 16:14
Copy link
Contributor

@cryptodev-2s cryptodev-2s left a comment

Choose a reason for hiding this comment

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

LGTM!

@Gudahtt Gudahtt enabled auto-merge (squash) August 20, 2025 16:21
@Gudahtt Gudahtt merged commit 60b318e into main Aug 20, 2025
231 checks passed
@Gudahtt Gudahtt deleted the remove-is-base-controller branch August 20, 2025 16:28
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.

[base-controller] Remove isBaseController

2 participants