Skip to content
This repository has been archived by the owner on Dec 9, 2024. It is now read-only.

Commit

Permalink
fix: Updated publishing script and only run tag on the main repo, not…
Browse files Browse the repository at this point in the history
… forks (#665)

* Only run tag if new version on the main repo, not on forks

* fixing GitHub tag name for GitHub release publishing

* add missing build step to publish script

* updated build step
  • Loading branch information
gligorkot authored May 16, 2023
1 parent dfeb0c5 commit 1caebed
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
1 change: 1 addition & 0 deletions .github/workflows/integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -194,6 +194,7 @@ jobs:
tag-if-new-version:
runs-on: ubuntu-latest
needs: [node-integration-tests, python-integration-tests]
if: github.repository_owner == 'serverless'

steps:
- name: Checkout repository
Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,9 @@ jobs:
run: |
npm ci
- name: Build local package
run: npm run build

- name: Publish new version
# Note: Setting NODE_AUTH_TOKEN as job|workspace wide env var won't work
# as it appears actions/setup-node sets own value
Expand All @@ -54,6 +57,6 @@ jobs:
- name: Create and publish a release
uses: softprops/action-gh-release@v1
with:
tag_name: ${GITHUB_REF##*/}
tag_name: ${{ github.ref_name }}
prerelease: false
generate_release_notes: true

0 comments on commit 1caebed

Please sign in to comment.