Skip to content

Removing unnecesary var#1634

Merged
siri-varma merged 1 commit into
dapr:masterfrom
salaboy:remove-old-var
Jan 30, 2026
Merged

Removing unnecesary var#1634
siri-varma merged 1 commit into
dapr:masterfrom
salaboy:remove-old-var

Conversation

@salaboy

@salaboy salaboy commented Jan 30, 2026

Copy link
Copy Markdown
Contributor

Description

Fixing CI and removing variable that was not needed.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

@salaboy
salaboy requested review from a team as code owners January 30, 2026 16:10
@salaboy salaboy mentioned this pull request Jan 30, 2026
3 tasks
@siri-varma
siri-varma merged commit 3ce6863 into dapr:master Jan 30, 2026
10 of 11 checks passed
@codecov

codecov Bot commented Jan 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.99%. Comparing base (fd16fe2) to head (5d6e41c).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1634      +/-   ##
============================================
- Coverage     79.02%   78.99%   -0.04%     
+ Complexity     2097     2096       -1     
============================================
  Files           228      228              
  Lines          6321     6321              
  Branches        693      693              
============================================
- Hits           4995     4993       -2     
- Misses          975      976       +1     
- Partials        351      352       +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

javier-aliaga pushed a commit to javier-aliaga/java-sdk that referenced this pull request Feb 2, 2026
javier-aliaga pushed a commit to javier-aliaga/java-sdk that referenced this pull request Feb 2, 2026
Signed-off-by: salaboy <[email protected]>
Signed-off-by: Javier Aliaga <[email protected]>
javier-aliaga pushed a commit to javier-aliaga/java-sdk that referenced this pull request Feb 2, 2026
javier-aliaga pushed a commit to javier-aliaga/java-sdk that referenced this pull request Feb 2, 2026
Signed-off-by: salaboy <[email protected]>
Signed-off-by: Javier Aliaga <[email protected]>
salaboy added a commit that referenced this pull request Feb 2, 2026
* push removing unnecesary var (#1634)

Signed-off-by: salaboy <[email protected]>
Signed-off-by: Javier Aliaga <[email protected]>

* Replace old log based waits with DaprWait strategy. (#1635)

* Replace old log based waits with DaprWait strategy.

Signed-off-by: Artur Ciocanu <[email protected]>

* Revert ConsumerAppIT to make tests pass.

Signed-off-by: Artur Ciocanu <[email protected]>

* Revert ConsumerAppIT so there are no changes.

Signed-off-by: Artur Ciocanu <[email protected]>

* Disable the IT test for failing CI.

Signed-off-by: Artur Ciocanu <[email protected]>

---------

Signed-off-by: Artur Ciocanu <[email protected]>
Signed-off-by: Javier Aliaga <[email protected]>

* Add PATCH to HttpMethods enum for service invocation (#1623)

* Add PATCH to HttpMethods enum for service invocation

Adds PATCH to the HttpMethods enum to enable using DaprClient.invokeMethod()
with PATCH requests through HTTPEndpoints. This is commonly required for
partial updates when calling external REST APIs via Dapr service invocation.

Fixes #1622

Signed-off-by: Paul Lindner <[email protected]>

* Add test for PATCH HTTP method

Signed-off-by: Paul Lindner <[email protected]>

---------

Signed-off-by: Paul Lindner <[email protected]>
Co-authored-by: salaboy <[email protected]>
Signed-off-by: Javier Aliaga <[email protected]>

---------

Signed-off-by: salaboy <[email protected]>
Signed-off-by: Javier Aliaga <[email protected]>
Signed-off-by: Artur Ciocanu <[email protected]>
Signed-off-by: Paul Lindner <[email protected]>
Co-authored-by: salaboy <[email protected]>
Co-authored-by: artur-ciocanu <[email protected]>
Co-authored-by: Paul Lindner <[email protected]>
@nelson-parente nelson-parente added this to the v1.17 milestone Feb 12, 2026
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