Skip to content

Conversation

@80avin
Copy link
Owner

@80avin 80avin commented Sep 7, 2025

dpetka2001 and others added 10 commits May 8, 2025 20:17
suggestions by @williamboman and @TheDSCPL

Co-authored-by: William Boman <[email protected]>
Co-authored-by: Luis Durão <[email protected]>
Take into account when `server_opts.enabled = false` to add the server
into `exclude_automatic_enable`, since `configure(server)` won't execute
for that.

Also `return true` for `opts.setup` to be in accordance with the rest of
`configure` function and remove `server_opts.enabled` conditional check
from `configure`, since it will only execute when `server_opts.enabled
~= false` either way.
@80avin 80avin merged commit ee238aa into 80avin:main Sep 7, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants