Skip to content

Dedup import path#8555

Merged
lovincyrus merged 1 commit intomainfrom
cyrus/dup-import-prettier
Dec 22, 2025
Merged

Dedup import path#8555
lovincyrus merged 1 commit intomainfrom
cyrus/dup-import-prettier

Conversation

@lovincyrus
Copy link
Copy Markdown
Contributor

@lovincyrus lovincyrus commented Dec 22, 2025 •
edited
Loading

This PR fixes a prettier bug that managed to slip into the latest main (#8480)

e.g. https://github.com/rilldata/rill/actions/runs/20428768897/job/58694645005?pr=8467

CleanShot 2025-12-22 at 18 24 33@2x

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@lovincyrus lovincyrus self-assigned this Dec 22, 2025
@lovincyrus lovincyrus marked this pull request as ready for review December 22, 2025 10:25
@lovincyrus lovincyrus merged commit eac103b into main Dec 22, 2025
15 of 16 checks passed
@lovincyrus lovincyrus deleted the cyrus/dup-import-prettier branch December 22, 2025 13:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ericpgreen2 ericpgreen2 Awaiting requested review from ericpgreen2

1 more reviewer

@mindspank mindspank mindspank approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@lovincyrus lovincyrus

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lovincyrus @mindspank