Skip to content

Bump MassTransit and fix missing method#686

Merged
sivvak merged 2 commits into
v8.1-previewfrom
mass-transit-bump
Aug 13, 2024
Merged

Bump MassTransit and fix missing method#686
sivvak merged 2 commits into
v8.1-previewfrom
mass-transit-bump

Conversation

@sivvak
Copy link
Copy Markdown
Contributor

@sivvak sivvak commented Aug 12, 2024

HasConsumerInterfaces method got deleted 🙈
Using IsConsumer instead seems reasonable 😃

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 12, 2024

Test Results

 35 files  ±0  136 suites  ±0   1m 5s ⏱️ ±0s
772 tests ±0  759 ✅ ±0  13 💤 ±0  0 ❌ ±0 
788 runs  ±0  768 ✅ ±0  20 💤 ±0  0 ❌ ±0 

Results for commit 46a2b4e. ± Comparison against base commit 9d9ac19.

♻️ This comment has been updated with latest results.

@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.61%. Comparing base (e205adb) to head (cbf2515).
Report is 6 commits behind head on v8.1-preview.

Additional details and impacted files
@@              Coverage Diff              @@
##           v8.1-preview     #686   +/-   ##
=============================================
  Coverage         65.61%   65.61%           
=============================================
  Files               263      263           
  Lines              5991     5991           
  Branches            377      377           
=============================================
  Hits               3931     3931           
  Misses             1948     1948           
  Partials            112      112           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Contributor

@Dragemil Dragemil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm doing it like this in my project and it seems to be fine.

@sivvak
Copy link
Copy Markdown
Contributor Author

sivvak commented Aug 13, 2024

I changed the way Postgres DB gets configured due to the problems occuring after MassTransit bump. After some research I discovered potential source of the mentioned problems described on SO. It looks like everything is working fine but please correct me if I am wrong 😅

@sivvak sivvak requested a review from Dragemil August 13, 2024 08:52
@sivvak sivvak merged commit b92f5c3 into v8.1-preview Aug 13, 2024
@sivvak sivvak deleted the mass-transit-bump branch August 13, 2024 09:01
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