Skip to content

feat: add support for global config flag#698

Merged
tconbeer merged 2 commits into
tconbeer:mainfrom
bradykieffer:feat/issue-671/add-config-path-option
Jul 28, 2025
Merged

feat: add support for global config flag#698
tconbeer merged 2 commits into
tconbeer:mainfrom
bradykieffer:feat/issue-671/add-config-path-option

Conversation

@bradykieffer

Copy link
Copy Markdown
Contributor

Pass --config to manually set a config file. I've opted to assert that this file exists and skip any searching for a pyproject.toml otherwise. You can also set the config file via the SQLFMT_CONFIG env var.

Closes: #671

Pass `--config` to manually set a config file. I've opted to assert that
this file exists and skip any searching for a pyproject.toml otherwise.
You can also set the config file via the `SQLFMT_CONFIG` env var.
@tconbeer

Copy link
Copy Markdown
Owner

thanks so much for this PR! I've been traveling and am working to catch up on everything now -- will try to get to this in the next week.

@tconbeer tconbeer left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

This looks amazing - thanks so much for the thorough implementation and tests!

Could you please add an entry to CHANGELOG.md? Then we can merge and ship this.

@bradykieffer

Copy link
Copy Markdown
Contributor Author

Sounds good @tconbeer! I can also push an update to the docs repo if you'd like

@tconbeer

Copy link
Copy Markdown
Owner

Yes, that would be incredibly helpful. Thank you so much

bradykieffer added a commit to bradykieffer/sqlfmt-docs that referenced this pull request Jul 28, 2025
@bradykieffer

Copy link
Copy Markdown
Contributor Author

Perfect! That's in #698

@tconbeer tconbeer merged commit 6d70122 into tconbeer:main Jul 28, 2025
15 of 16 checks passed
@tconbeer

Copy link
Copy Markdown
Owner

Thanks, @bradykieffer !

tconbeer pushed a commit to tconbeer/sqlfmt-docs that referenced this pull request Jul 28, 2025
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.

Add config path option

2 participants