Skip to content

Commit d042180

Browse files
Final push for fixing the build step
Signed-off-by: Xavier Geerinck <[email protected]>
1 parent 726642a commit d042180

File tree

1 file changed

+12
-12
lines changed

1 file changed

+12
-12
lines changed

.github/workflows/build.yml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -41,35 +41,35 @@ jobs:
4141
run: ./scripts/build.sh
4242

4343
# @TODO: add a depend on the test-e2e pipeline?
44-
# - name: Run unit tests
45-
# id: tests
46-
# run: npm run test:unit:all
44+
- name: Run unit tests
45+
id: tests
46+
run: npm run test:unit:all
4747

48-
# - name: Upload test coverage
49-
# uses: codecov/codecov-action@v1
48+
- name: Upload test coverage
49+
uses: codecov/codecov-action@v1
5050

51-
# - name: Is Release?
52-
# if: startswith(github.ref, 'refs/tags/v')
53-
# run: echo "DEPLOY_PACKAGE=true" >> $GITHUB_ENV
51+
- name: Is Release?
52+
if: startswith(github.ref, 'refs/tags/v')
53+
run: echo "DEPLOY_PACKAGE=true" >> $GITHUB_ENV
5454

5555
# note: package.json gets updated here for the new package name
5656
- name: Publish to npm (@dapr/dapr)
57-
# if: env.DEPLOY_PACKAGE == 'true'
57+
if: env.DEPLOY_PACKAGE == 'true'
5858
run: npm publish --access public
5959
env:
6060
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
6161

6262
- name: "[dapr-client] Configure to publish to dapr-client for deprecation notice reasons"
63-
# if: env.DEPLOY_PACKAGE == 'true'
63+
if: env.DEPLOY_PACKAGE == 'true'
6464
run: sed -i s#"@dapr/dapr"#"dapr-client"# package.json
6565

6666
- name: "[dapr-client] Build Package"
67-
# if: env.DEPLOY_PACKAGE == 'true'
67+
if: env.DEPLOY_PACKAGE == 'true'
6868
run: ./scripts/build.sh
6969

7070
# note: package.json gets updated here for the new package name
7171
- name: "[dapr-client] Publish to npm (dapr-client)"
72-
# if: env.DEPLOY_PACKAGE == 'true'
72+
if: env.DEPLOY_PACKAGE == 'true'
7373
run: npm publish --access public
7474
env:
7575
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

0 commit comments

Comments
 (0)