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

V2 #34

Merged
merged 6 commits into from
Jun 24, 2024
Merged

V2 #34

merged 6 commits into from
Jun 24, 2024

Conversation

nicoandrewss
Copy link
Contributor

BREAKING CHANGE:

  • the eventbus now has to be started separately after the declaration

Why?: when used in services/applications, the main setup might has not yet fully finished initializing, but the added handlers already handle events from the eventbus, which can lead to panics, and even the app not starting at all.

Note: the eventbus can be switched back to the previous behaviour by using the
WithHandlerConsumeAfterAdd() option while declaring.

Due to the breaking change:

  • bumps up the module version to v2

Also:

  • updates tests
  • updates linter config
  • updates actions workflow

@hanspolo
Copy link
Member

Wir (Menschen im Büro) fragen uns gerade, ob es nicht hilfreich wäre, noch eine Funktion anzubieten, mit der man eine Liste von Handlern anhängen kann und dann auch gleich im Anschluss das Handling zu starten.
Das würde das Initialisieren ggf. ein bisschen übersichtlicher machen.

@nicoandrewss
Copy link
Contributor Author

Wir (Menschen im Büro) fragen uns gerade, ob es nicht hilfreich wäre, noch eine Funktion anzubieten, mit der man eine Liste von Handlern anhängen kann und dann auch gleich im Anschluss das Handling zu starten.

Das würde das Initialisieren ggf. ein bisschen übersichtlicher machen.

Ah, ja sowas hab ich mir tatsächlich schonmal gebaut! Kann ich einbauen, gute Idee 😬

@nicoandrewss nicoandrewss merged commit d885c50 into main Jun 24, 2024
3 checks passed
@nicoandrewss nicoandrewss deleted the V2 branch June 24, 2024 11:33
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.

3 participants