Conversation
High-level overviewThis PR standardizes the Prettier version used across DataFusion’s documentation-related scripts. During the review, I identified several scripts in both The chosen approach keeps the solution intentionally simple:
This provides consistent formatting behavior across local workflows and CI, while keeping the change minimal and easy to review. As a future improvement, the Prettier version could be centralized (for example, via a shared script or config) if additional tooling versions need to be managed in a similar way. For now, keeping the version local to each script avoids adding new dependencies and keeps the scope focused. |
|
@Jefffrey since you originally pointed this #19750 (comment), it would be great if you could take a look at this PR. Your feedback or any improvements would be much appreciated. |
Jefffrey
left a comment
There was a problem hiding this comment.
Would be nice if we figured a way to unify these, but aligning them is a good first step; perhaps leave the original issue open for this?
|
@Jefffrey Thanks! Yes, this PR focuses on aligning the Prettier versions with a minimal change. I agree that fully unifying the configuration (e.g. centralizing the version) would be a good next step, and I plan to continue working on that as a follow-up. Leaving the original issue open makes sense to me. |
Which issue does this PR close?
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?