Skip to content

pin prettier version and fix formatting of file affected by minor or … - #51

Merged
Ethan-Arrowood merged 2 commits into
mainfrom
pin-prettier
Dec 3, 2025
Merged

pin prettier version and fix formatting of file affected by minor or …#51
Ethan-Arrowood merged 2 commits into
mainfrom
pin-prettier

Conversation

@Ethan-Arrowood

Copy link
Copy Markdown
Member

…patch change

This started randomly affecting my integration test PR.

@Ethan-Arrowood
Ethan-Arrowood requested a review from a team as a code owner December 2, 2025 22:41

@kriszyp kriszyp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The dependency range properly alerted us to the issue. This PR fixes the issue. I don't think the dependency needs to be pinned.

@kriszyp

kriszyp commented Dec 3, 2025

Copy link
Copy Markdown
Member

How about a compromise and let's make it a ~ patch-only version range?

@Ethan-Arrowood

Copy link
Copy Markdown
Member Author

I'm happy with that compromise!

@kriszyp kriszyp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@Ethan-Arrowood
Ethan-Arrowood merged commit fb098c0 into main Dec 3, 2025
7 checks passed
@Ethan-Arrowood
Ethan-Arrowood deleted the pin-prettier branch December 3, 2025 18:26
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.

4 participants