Skip to content

fix(components): Cleanup circular dep in Autocomplete#2861

Open
scotttjob wants to merge 3 commits intomasterfrom
CLEANUP/circular-dep
Open

fix(components): Cleanup circular dep in Autocomplete#2861
scotttjob wants to merge 3 commits intomasterfrom
CLEANUP/circular-dep

Conversation

@scotttjob
Copy link
Contributor

Motivations

Noticed a circular dependency in Autocomplete.

Changes

Moved the circular type to the already created type file, and imported from there.

Testing

  1. Everything should work as-is.
  2. We should no longer have a circular dep in components.

Changes can be
tested via Pre-release


In Atlantis we use Github's built in pull request reviews.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Jan 5, 2026

Deploying atlantis with  Cloudflare Pages  Cloudflare Pages

Latest commit: f3c2f57
Status: ✅  Deploy successful!
Preview URL: https://b6c6ceb7.atlantis.pages.dev
Branch Preview URL: https://cleanup-circular-dep.atlantis.pages.dev

View logs

Copy link
Contributor

@MichaelParadis MichaelParadis left a comment

Choose a reason for hiding this comment

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

LGTM!

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

Development

Successfully merging this pull request may close these issues.

2 participants