Skip to content

Move config api to stable#1077

Merged
halspang merged 5 commits into
dapr:masterfrom
shivamkm07:stable_configapi
May 3, 2023
Merged

Move config api to stable#1077
halspang merged 5 commits into
dapr:masterfrom
shivamkm07:stable_configapi

Conversation

@shivamkm07
Copy link
Copy Markdown
Contributor

Description

This PR updates config API to stable. It updates the protos to include the stable apis. There has been some workflow changes causing the build to fail with latest dapr protos. It also fixes it

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

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
@shivamkm07 shivamkm07 requested review from a team as code owners April 21, 2023 12:24
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 21, 2023

Codecov Report

Merging #1077 (20be985) into master (9b470cc) will decrease coverage by 0.02%.
The diff coverage is 60.00%.

@@            Coverage Diff             @@
##           master    #1077      +/-   ##
==========================================
- Coverage   68.95%   68.93%   -0.02%     
==========================================
  Files         169      169              
  Lines        5534     5534              
  Branches      588      588              
==========================================
- Hits         3816     3815       -1     
- Misses       1578     1579       +1     
  Partials      140      140              
Flag Coverage Δ
net6 68.86% <60.00%> (ø)
net7 68.86% <60.00%> (ø)
netcoreapp3.1 68.90% <60.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/Dapr.Client/DaprClientGrpc.cs 81.73% <60.00%> (-0.13%) ⬇️

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
@shivamkm07
Copy link
Copy Markdown
Contributor Author

@dapr/maintainers-dotnet-sdk @dapr/approvers-dotnet-sdk Pinging for reviews.

Copy link
Copy Markdown
Contributor

@halspang halspang left a comment

Choose a reason for hiding this comment

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

Two small comments.

Comment thread .github/workflows/itests.yml Outdated
Comment thread src/Dapr.Client/DaprClientGrpc.cs
Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
@halspang halspang merged commit 3d4ae5b into dapr:master May 3, 2023
onionhammer pushed a commit to onionhammer/dotnet-sdk that referenced this pull request May 30, 2023
* updating protos

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>

* updating config api to stable

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>

* workflow changes to fix build failure

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>

* updating go  version

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>

* go version from dapr/cli repo

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>

---------

Signed-off-by: Shivam Kumar <shivamkm07@gmail.com>
Signed-off-by: Erik O'Leary <erik.m.oleary@gmail.com>
@halspang halspang added this to the v1.11 milestone Jun 1, 2023
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