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

Fixing a bug in the starter until the workflow is correctly construct… #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Roustalski
Copy link

@Roustalski Roustalski commented Oct 21, 2021

We were getting ready to write an enterprise bus messaging queue from scratch because we didn't think anything existed for TS already, but we were excited to find node-ts/*!

Anyway, we saw that you recently removed InversifyJS as a hard dependency. Sad, but I understand the reasoning (We just InversifyJS extensively). The removal caused an issue with the starter, so I temporarily setup the bus with a container using Inversify in main until you guys work out how to get a bus instance to the workflows internally.

@adenhertog
Copy link
Contributor

Hey @Roustalski - thanks heaps for raising this!

The choice to remove inversify was a tough one, but I felt it's better to let developers choose if they want to use DI and if so choose the DI lib they're comfortable with.

I've fixed the bus reference in #27 and added a much needed test in an effort to prevent this library forcing the use of inversify on everyone.

Cheers

@adenhertog
Copy link
Contributor

Hey @Roustalski I tried responding to your email but looks like there's a problem with the server

Screenshot_2021-10-31-15-49-18-970_com google android gm

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