Skip to content

Commit

Permalink
Merge updates from master into feature/new_quickstarts (#573)
Browse files Browse the repository at this point in the history
* 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](follow-redirects/follow-redirects@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](follow-redirects/follow-redirects@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>
  • Loading branch information
9 people committed Mar 4, 2022
1 parent 90a125f commit 3158377
Show file tree
Hide file tree
Showing 13 changed files with 53,284 additions and 60 deletions.
10 changes: 0 additions & 10 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,6 @@ on:
- v*
paths-ignore:
- '**.md'
pull_request:
branches:
- master
- feature/new_quickstarts
- release-*
workflow_dispatch:
jobs:
build:
Expand Down Expand Up @@ -69,11 +64,9 @@ jobs:
popd
done
- name: docker login
if: github.event_name != 'pull_request'
run: |
docker login -u ${{ secrets.DOCKER_REGISTRY_ID }} -p ${{ secrets.DOCKER_REGISTRY_PASS }}
- name: publish image to dockerhub
if: github.event_name != 'pull_request'
run: |
SAMPLE_LIST=(tutorials/hello-kubernetes tutorials/distributed-calculator tutorials/pub-sub tutorials/bindings tutorials/observability tutorials/secretstore)
Expand All @@ -95,7 +88,6 @@ jobs:
- name: Parse release version and set REL_VERSION
run: python ./.github/scripts/get_release_version.py
- name: create docker manifest
if: github.event_name != 'pull_request'
run: |
SAMPLE_LIST=(tutorials/hello-kubernetes tutorials/distributed-calculator tutorials/pub-sub tutorials/bindings tutorials/observability tutorials/secretstore)
Expand All @@ -106,11 +98,9 @@ jobs:
popd
done
- name: docker login
if: github.event_name != 'pull_request'
run: |
docker login -u ${{ secrets.DOCKER_REGISTRY_ID }} -p ${{ secrets.DOCKER_REGISTRY_PASS }}
- name: publish manifest to dockerhub
if: github.event_name != 'pull_request'
run: |
SAMPLE_LIST=(tutorials/hello-kubernetes tutorials/distributed-calculator tutorials/pub-sub tutorials/bindings tutorials/observability tutorials/secretstore)
Expand Down
26,596 changes: 26,596 additions & 0 deletions distributed-calculator/react-calculator/client/package-lock.json

Large diffs are not rendered by default.

Loading

0 comments on commit 3158377

Please sign in to comment.