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

[dotnet-client] Add dist lock examples #1095

Merged
merged 5 commits into from
May 22, 2023
Merged

Conversation

hhunter-ms
Copy link
Contributor

Description

Add missing dist lock examples to the dotnet client doc

Issue reference

PR will help close: dapr/docs#2770

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

@hhunter-ms hhunter-ms requested review from a team as code owners May 19, 2023 15:42
@hhunter-ms hhunter-ms mentioned this pull request May 19, 2023
11 tasks
@codecov
Copy link

codecov bot commented May 19, 2023

Codecov Report

Merging #1095 (e354552) into master (e59c856) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1095   +/-   ##
=======================================
  Coverage   67.49%   67.49%           
=======================================
  Files         170      170           
  Lines        5652     5652           
  Branches      600      600           
=======================================
  Hits         3815     3815           
  Misses       1697     1697           
  Partials      140      140           
Flag Coverage Δ
net6 67.42% <ø> (ø)
net7 67.42% <ø> (ø)
netcoreapp3.1 67.46% <ø> (ø)

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

Copy link
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.

Just some nitpicks.

Comment on lines 217 to 218
string DAPR_LOCK_NAME = "lockstore";
string fileName = "my_file_name";
Copy link
Contributor

Choose a reason for hiding this comment

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

We should be consistent in our variable naming scheme.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@halspang let me know if I addressed this right

Comment on lines 249 to 250
string DAPR_LOCK_NAME = "lockstore";
var client = new DaprClientBuilder().Build();
Copy link
Contributor

Choose a reason for hiding this comment

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

Same.

Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Hannah Hunter <[email protected]>
@halspang halspang merged commit 364ed92 into dapr:master May 22, 2023
onionhammer pushed a commit to onionhammer/dotnet-sdk that referenced this pull request May 30, 2023
* add dist lock to dotnet client doc

Signed-off-by: Hannah Hunter <[email protected]>

* attempt at naming scheme 1

Signed-off-by: Hannah Hunter <[email protected]>

* yikes put it back

Signed-off-by: Hannah Hunter <[email protected]>

* attempt 2

Signed-off-by: Hannah Hunter <[email protected]>

* attempt 3

Signed-off-by: Hannah Hunter <[email protected]>

---------

Signed-off-by: Hannah Hunter <[email protected]>
Signed-off-by: Erik O'Leary <[email protected]>
@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.

SDK documentation gaps
2 participants