Skip to content

Commit 69e8627

Browse files
Apply suggestions from code review
Co-authored-by: Mickael Bourgois <35171168+BourgoisMickael@users.noreply.github.com>
1 parent 240ff79 commit 69e8627

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

.github/workflows/release.yaml

+3-2
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ jobs:
3131
echo PKG_BASENAME="$(basename $(jq -r .name < package.json))" >> $GITHUB_ENV
3232
- name: Ensure the branch is protected
3333
run: |
34-
if [[ "${{ github.ref_protected }}" == "false" ]]; then
34+
if [[ "${{ github.ref_protected }}" = "false" ]]; then
3535
echo "::error::The branch ${{ github.ref }} is not protected"
3636
exit 1
3737
fi
@@ -54,7 +54,8 @@ jobs:
5454
cache: yarn
5555
node-version: '16'
5656
registry-url: https://npm.pkg.github.com
57-
- run: yarn install --frozen-lockfile
57+
- name: install dependencies
58+
run: yarn install --frozen-lockfile
5859
- name: Publish to GitHub Packages
5960
run: npm publish --provenance
6061
env:

.github/workflows/tests.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ jobs:
2929
run: yarn lint
3030
- name: Set common env vars for npm publish
3131
run: |
32-
echo "SHORTSHA=$(echo $GITHUB_SHA | cut -c1-7)" >> $GITHUB_ENV
32+
echo "SHORTSHA=$(git rev-parse --short $GITHUB_SHA)" >> $GITHUB_ENV
3333
echo "PKG_VERSION=$(jq -r .version < package.json)" >> $GITHUB_ENV
3434
- name: Set version for work in progress
3535
run: yarn version --no-git-tag-version --new-version "0.0.0-sha.${SHORTSHA}"

0 commit comments

Comments
 (0)