Skip to content

Fix name in dicussion.md - #1026

Merged
halspang merged 3 commits into
dapr:masterfrom
tmacam:patch-1
Feb 22, 2023
Merged

halspang merged 3 commits into
dapr:masterfrom
tmacam:patch-1

Conversation

@tmacam

@tmacam tmacam commented Feb 7, 2023 •

Copy link
Copy Markdown
Contributor

Description

dicussion.md name field was colliding with feature_request.md name, causing both to be unusable:

Changing the name on discussion.md should fix the problem.

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

dicussion.md `name` field was colliding with feature_request.md `name`, causing both to be unusable. 

Signed-off-by: Tiago Alves Macambira <tmacam@burocrata.org>
@codecov

codecov Bot commented Feb 7, 2023 •

Copy link
Copy Markdown

Codecov Report

Merging #1026 (e948bc7) into master (aa38b05) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1026      +/-   ##
==========================================
- Coverage   69.47%   69.45%   -0.02%     
==========================================
  Files         162      162              
  Lines        5432     5432              
  Branches      585      585              
==========================================
- Hits         3774     3773       -1     
- Misses       1518     1519       +1     
  Partials      140      140              
Flag Coverage Δ
net6 69.38% <ø> (ø)
net7 69.38% <ø> (ø)
netcoreapp3.1 69.42% <ø> (-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% <0.00%> (-0.13%) ⬇️

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

@msfussell msfussell left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@halspang
halspang merged commit 5ece9d0 into dapr:master Feb 22, 2023
@tmacam
tmacam deleted the patch-1 branch February 22, 2023 02:43
yash-nisar pushed a commit to yash-nisar/dotnet-sdk that referenced this pull request Feb 27, 2023
dicussion.md `name` field was colliding with feature_request.md `name`, causing both to be unusable.

Signed-off-by: Tiago Alves Macambira <tmacam@burocrata.org>
@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.

3 participants