Skip to content

Conversation

@fpseverino
Copy link
Member

As @czechboy0 said:

[...] now that ConsoleKit uses Swift Configuration, I think this should be renamed to something Vapor/ConsoleKit-specific: LoggingSystem.bootstrap(config: ConfigReader). As this is API that could presumably be added to Swift Log/Swift Configuration themselves, it's kind of similar to the retroactive conformance feature.

Eg ConsoleLogger.bootstrap(config:) or something would be fine.

I have renamed it according to the above suggestion. If anyone has any other ideas, please let me know!

@fpseverino fpseverino merged commit 772b4c8 into vapor:consolekit-5 Dec 12, 2025
13 checks passed
@fpseverino fpseverino deleted the rename-bootstrap branch December 12, 2025 18:00
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.

2 participants