Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move the default Configuration.init() into a separate file. #580

Merged
merged 1 commit into from Aug 7, 2023

Conversation

allevato
Copy link
Collaborator

@allevato allevato commented Aug 4, 2023

This is meant to make it easier for users forking or building their own swift-format to change the hardcoded default configuration.

This also adds a helper for the default test configuration. We can't let the unit tests rely on the default configuration because that would make the tests fail when run with a swift-format binary that has a different default configuration. This approach lets the default test configuration differ from the "real" default configuration.

This is meant to make it easier for users forking or building
their own swift-format to change the hardcoded default
configuration.

This also adds a helper for the default test configuration. We
can't let the unit tests rely on the default configuration
because that would make the tests fail when run with a
swift-format binary that has a different default configuration.
This approach lets the default test configuration differ from
the "real" default configuration.
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.

None yet

1 participant