Skip to content

chore(dev-deps): update eslint-config-prettier to v10 - #493

Merged
G-Rath merged 1 commit into
mainfrom
update-dependencies
Sep 12, 2026
Merged

G-Rath merged 1 commit into
mainfrom
update-dependencies

Conversation

@MichaelDeBoey

Copy link
Copy Markdown
Member

Mainly updating eslint-config-prettier to v10

@MichaelDeBoey MichaelDeBoey added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Sep 12, 2026
@G-Rath

G-Rath commented Sep 12, 2026 •

Copy link
Copy Markdown
Collaborator

what's the gain in bumping all the versions in the package.json? we'll already be using the latest minor versions in CI since we don't have a lockfile

@MichaelDeBoey

Copy link
Copy Markdown
Member Author

@G-Rath I figured, I'd just update all others as well, since I was already at this file 🤷‍♂️

@G-Rath

G-Rath commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

I think let's just bump eslint-config-prettier and call it a day

@MichaelDeBoey MichaelDeBoey changed the title chore: update dependencies chore(dev-deps): update eslint-config-prettier to v10 Sep 12, 2026
@MichaelDeBoey

Copy link
Copy Markdown
Member Author

@G-Rath It doesn't matter that other deps are updated as well, but fine 🤷‍♂️

@G-Rath

G-Rath commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

@MichaelDeBoey it doesn't matter that much but it does make the diff + blame a bit cleaner, and better reflects the fact that we don't require those latest versions specifically especially for dependencies like eslint and typescript.

I appreciate you indulging my preference :)

@G-Rath
G-Rath merged commit dea2896 into main Sep 12, 2026
91 checks passed
@G-Rath
G-Rath deleted the update-dependencies branch September 12, 2026 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants