Skip to content

fix(treefmt): require config file#777

Merged
stevearc merged 1 commit intostevearc:masterfrom
zivarah:zivarah/treefmt-require-config
Sep 27, 2025
Merged

fix(treefmt): require config file#777
stevearc merged 1 commit intostevearc:masterfrom
zivarah:zivarah/treefmt-require-config

Conversation

@zivarah
Copy link
Contributor

@zivarah zivarah commented Sep 22, 2025

Running treefmt when no config file exists just results in an error, so checking for the presence of a config file by default makes sense.

This will allow treefmt to be included as a formatter that should be run on all file types, without throwing errors for projects that have no treefmt config file.

Running `treefmt` when no config file exists just results in an error,
so checking for the presence of a config file by default makes sense.

This will allow `treefmt` to be included as a formatter that should be
run on all file types, without throwing errors for projects that have no
treefmt config file.
@github-actions github-actions bot requested a review from stevearc September 22, 2025 06:20
@stevearc
Copy link
Owner

Thanks for the PR!

@stevearc stevearc merged commit 3bd9996 into stevearc:master Sep 27, 2025
14 of 17 checks passed
@DrTeagle
Copy link
Contributor

I think this PR is what is causing my issue, should we maybe revert or consider just checking if treefmt runs successfully?

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.

3 participants

Comments