Skip to content

Commit

Permalink
Merge pull request #2219 from llimeht/tmp/no-pr-releases
Browse files Browse the repository at this point in the history
After discussions, it is still felt that a proper solution is still needed (beyond the default) however this is not that used currently and solves a number of other issues in PR's currently so agree to merge this.
  • Loading branch information
butlerpd authored Sep 27, 2022
2 parents ee4bd64 + 3f4d0c7 commit 09e96e6
Showing 1 changed file with 0 additions and 24 deletions.
24 changes: 0 additions & 24 deletions .github/workflows/installers.yml
Original file line number Diff line number Diff line change
Expand Up @@ -194,27 +194,3 @@ jobs:
name: SasView-Installer-${{ matrix.os }}-${{ matrix.python-version }}-${{ github.sha }}
path: installers/dist/SasView5.dmg
if-no-files-found: error

- name: Declare env variables on push only
if: github.event_name == 'push'
shell: bash
run: echo "BRANCH_NAME=${GITHUB_REF#refs/*/}" >> $GITHUB_ENV

- name: Declare env variables on pull_request only
if: github.event_name == 'pull_request'
shell: bash
run: echo "BRANCH_NAME=$GITHUB_HEAD_REF" >> $GITHUB_ENV

- name: Upload installer to GitHub releases
#Release and main branch will be treated differently
if: env.BRANCH_NAME != 'main'
uses: ncipollo/release-action@v1
with:
draft: true
prerelease: true
allowUpdates: true
replacesArtifacts: true
token: ${{ secrets.GITHUB_TOKEN }}
artifacts: "installers/dist/SasView5.dmg, installers/Output/setupSasView.exe"
name: ${{ env.BRANCH_NAME }}
tag: ${{ env.BRANCH_NAME }}

0 comments on commit 09e96e6

Please sign in to comment.