-
Notifications
You must be signed in to change notification settings - Fork 530
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
Moving middleware out of repo to dapr/samples #513
Conversation
Signed-off-by: Paul Yuknewicz <[email protected]>
7af25de
to
a098291
Compare
Related to: dapr/samples#94 |
You also have to edit the https://github.com/dapr/quickstarts readme file, to remove the middleware from the list and the page |
@msfussell good catch, i did this from docs but not readme itself. thank you. |
Signed-off-by: Paul Yuknewicz <[email protected]>
442fa58
to
4473118
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Paul Yuknewicz <[email protected]>
All tests pass after re-run. @msfussell |
* Moving middleware out of repo to dapr/samples Signed-off-by: Paul Yuknewicz <[email protected]> * removing middleware from readme Signed-off-by: Paul Yuknewicz <[email protected]> * Updated test to remove middleware sample folder Signed-off-by: Paul Yuknewicz <[email protected]> Co-authored-by: Mark Fussell <[email protected]> Signed-off-by: Arash Rohani <[email protected]>
Description
Moving middleware sample out of this repo and into the dapr/samples repo
Issue reference
Fixes #512
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: