Some commands were unclear earlier. Changes to provide more clarity on them
Co-authored-by: Mark Fussell <markfussell@gmail.com>
Signed-off-by: shivanisinghnitp <58537821+shivanisinghnitp@users.noreply.github.com>
* Removed the extra space in expected test result
Signed-off-by: Arash Rohani <Arash.Rohani@gmail.com>
* Changed the output match from exact to substring to fix the issue.
Signed-off-by: Arash Rohani <Arash.Rohani@gmail.com>
* Replaced the logger.LogInformation with Console.WriteLine to fix the test issue.
Signed-off-by: Arash Rohani <Arash.Rohani@gmail.com>
* Fixed the whitespace issue and added sleep directive
Signed-off-by: Arash Rohani <Arash.Rohani@gmail.com>
* moving devcontainer to root to enable
* Fix for tutorials to point to new dapr-cli-link
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed additional broken links found by MMD `make validate` in tutorials.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changing readme to call python3 instead of python per best practices
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed zipkin api bug failing tests. Duration with current multiply code will never be slow enough to return traces results!
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Bump follow-redirects in /distributed-calculator/react-calculator/client (#530)
Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.7...v1.14.8)
---
updated-dependencies:
- dependency-name: follow-redirects
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Bump follow-redirects in /pub-sub/react-form/client (#531)
Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.14.7 to 1.14.8.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](https://github.com/follow-redirects/follow-redirects/compare/v1.14.7...v1.14.8)
---
updated-dependencies:
- dependency-name: follow-redirects
dependency-type: indirect
...
Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>
* Refresh pub-sub quickstart (tutorial) fixing routes and stale dependency (#539)
* Fixing error caused by route to * conflicing with /dapr/subscribe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrating from `requests` to `axios` to improve supportability
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changed from " to ' matching existing styles for Javascript
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changing `axios` from import to require('axios'), matching require('express')
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Consistent use of Async enabled http code
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed react * route bug that conflicts with /dapr/subscribe route
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Reducing log levels introduced by migration. added debug flag.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating MM test to match improved precision of Express server.js with Axios.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* updating console.log from console.debug (minor) for consistency.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updated vars to const, and then updated README to match.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Async posts needed await. Readme's tweaked to match updates.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Async should pair with await
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixing comment
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Removing commented out pipe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in. Updated dependencies
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Paulyuk/fixpubsuberror (#547)
* Fixing error caused by route to * conflicing with /dapr/subscribe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrating from `requests` to `axios` to improve supportability
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changed from " to ' matching existing styles for Javascript
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changing `axios` from import to require('axios'), matching require('express')
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Consistent use of Async enabled http code
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed react * route bug that conflicts with /dapr/subscribe route
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Reducing log levels introduced by migration. added debug flag.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating MM test to match improved precision of Express server.js with Axios.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* updating console.log from console.debug (minor) for consistency.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updated vars to const, and then updated README to match.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Async posts needed await. Readme's tweaked to match updates.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Async should pair with await
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixing comment
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Removing commented out pipe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in. Updated dependencies
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Hotfix to test, given precision sensitivity on test output number
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Improved MM test fix for quickstarts (#548)
* Fixing error caused by route to * conflicing with /dapr/subscribe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrating from `requests` to `axios` to improve supportability
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changed from " to ' matching existing styles for Javascript
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Changing `axios` from import to require('axios'), matching require('express')
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Consistent use of Async enabled http code
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Migrated distributed calculator from `request` with pipe to `axios` with service invoke.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixed react * route bug that conflicts with /dapr/subscribe route
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Reducing log levels introduced by migration. added debug flag.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating MM test to match improved precision of Express server.js with Axios.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* updating console.log from console.debug (minor) for consistency.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updated vars to const, and then updated README to match.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Updating Axios release(s) in Distributed Calc and pub-sub to v0.26 per @shubham1172 review.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Async posts needed await. Readme's tweaked to match updates.
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Update pub-sub/README.md
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Async should pair with await
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixing comment
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Removing commented out pipe
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review updated JS styles: return, casting, inlining URLs, and removing console.log
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Per @XavierGeerinck review migrated from BodyParser.json() middleware to Express.json() built in. Updated dependencies
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Hotfix to test, given precision sensitivity on test output number
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
* Fixing all remaining decimal precision issues with MM tests in readmes, including observability which now fails
Signed-off-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
* Fix curl sample to work on macOS (#543)
Signed-off-by: Marcin Jahn <marcinj10@gmail.com>
* Fix sample path in readme (#555)
Signed-off-by: Xin Shi <xshi@microsoft.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>
* Remove PR check for docker login (#565)
* remove PR check for docker login
Signed-off-by: tanvigour <tanvi.gour@gmail.com>
* remove PR trigger for the workflow to fix docker login issue
Signed-off-by: tanvigour <tanvi.gour@gmail.com>
Co-authored-by: Artur Souza <artursouza.ms@outlook.com>
* Test validation cleanup (#569)
* Test validation cleanup
Signed-off-by: tanvigour <tanvi.gour@gmail.com>
* add one response message in each of the validation steps
Signed-off-by: tanvigour <tanvi.gour@gmail.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Fussell <mfussell@microsoft.com>
Co-authored-by: Paul Yuknewicz <paulyuk@microsoft.com>
Co-authored-by: Shubham Sharma <shubhash@microsoft.com>
Co-authored-by: Marcin Jahn <marcinj10@gmail.com>
Co-authored-by: Xin Shi <xshi@microsoft.com>
Co-authored-by: tanvigour <60332928+tanvigour@users.noreply.github.com>
Co-authored-by: Artur Souza <artursouza.ms@outlook.com>
Paul Yuknewicz
shivanisinghnitp
Shivani Singh
Francesco Belacca
Amulya Varote
amulyavarote
Hal Spang
Arash Rohani
greenie-msft
Nick Greenfield