Skip to content

Conversation

ydessoukyy
Copy link
Contributor

@ydessoukyy ydessoukyy commented Sep 7, 2025

This PR updates the Initialize function to properly return errors from NewClient.
Previously, Initialize would just return immediately, leaving it unclear whether the client was successfully initialized or if there was a connection/configuration issue.

With this change, callers of Initialize can now handle initialization errors gracefully.

Important files

unleash.go (updated Initialize function)

closes #205
Discussion points

Should we also consider adding a health check (pinging the Unleash server) during initialization, or is propagating the error enough for now?

@FredrikOseberg FredrikOseberg merged commit 40bd141 into Unleash:v5 Sep 26, 2025
15 of 26 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Issues and PRs Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

unleash.Initialize() method does not return error if client creation fails
2 participants