Skip to content

Conversation

ItalyPaleAle
Copy link
Contributor

Fixes a regression (typo) introduced in #5648 . This is the reason for the test failures in components-contrib (see dapr/components-contrib#2405 )

Copy link
Member

@berndverst berndverst left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!

@berndverst
Copy link
Member

Currently dapr/dapr@master is broken and cannot be used in contrib for certification tests (causing a fatal error starting the profiling server). This fixes that.

Please merge as soon as possible @daixiang0 @yaron2 @artursouza

This will unblock others like @DeepanshuA

@codecov
Copy link

codecov bot commented Jan 5, 2023

Codecov Report

Merging #5707 (24d5cbf) into master (82b6903) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #5707      +/-   ##
==========================================
- Coverage   64.67%   64.64%   -0.03%     
==========================================
  Files         150      150              
  Lines       15734    15734              
==========================================
- Hits        10176    10172       -4     
- Misses       4834     4838       +4     
  Partials      724      724              
Impacted Files Coverage Δ
pkg/http/server.go 65.40% <0.00%> (ø)
pkg/components/pubsub/pluggable.go 64.63% <0.00%> (-4.88%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@daixiang0 daixiang0 merged commit 785d201 into dapr:master Jan 5, 2023
@DeepanshuA
Copy link
Contributor

Thanks @ItalyPaleAle @berndverst @daixiang0 for quick investigation, fix and merge. It really helps to move forward.

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.

4 participants