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

Update _index.md #552

Merged
merged 2 commits into from
Nov 20, 2023
Merged

Update _index.md #552

merged 2 commits into from
Nov 20, 2023

Conversation

fyzact
Copy link
Contributor

@fyzact fyzact commented Nov 18, 2023

fix how define dapr server to it can work directly when you copy

Description

Please explain the changes you've made

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

fix how define dapr server to it can work directly when you copy

Signed-off-by: Feyyaz Acet <[email protected]>
@fyzact fyzact requested review from a team as code owners November 18, 2023 15:04
@shubham1172
Copy link
Member

Hi, thanks for the PR, the code is self-explanatory there because of L104.

@fyzact
Copy link
Contributor Author

fyzact commented Nov 20, 2023

Hi, thanks for the PR, the code is self-explanatory there because of L104.

thanks for explanition, to me it was more aboout to fix the sample as it was not working directly when you copy.

@shubham1172
Copy link
Member

Oh I just noticed the comma missing, thanks for pointing it out.

@shubham1172 shubham1172 reopened this Nov 20, 2023
@shubham1172 shubham1172 added this pull request to the merge queue Nov 20, 2023
Merged via the queue into dapr:main with commit 2d41bc8 Nov 20, 2023
2 of 4 checks passed
@fyzact
Copy link
Contributor Author

fyzact commented Nov 20, 2023

Oh I just noticed the comma missing, thanks for pointing it out.

in fact, after pr I found too many issue like that, will fix all of them and raise new and more comprehensive pr next week hopefully.

@shubham1172
Copy link
Member

That would be awesome, thanks a lot!

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