Skip to content

Commit

Permalink
huh why is it not triggering
Browse files Browse the repository at this point in the history
  • Loading branch information
p-goulart committed Feb 5, 2024
1 parent ef76da2 commit 99693f6
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 5 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ jobs:
path: ${{ env.LT_HOME }}
token: ${{ secrets.LT_OS_TOKEN }}
# TODO: remove this once the new tokenisation branch is merged into master!
branch: pt/dict/new_tokenisation
ref: pt/dict/new_tokenisation

- name: Build LT
working-directory: ${{ env.LT_HOME }}
Expand Down Expand Up @@ -143,7 +143,7 @@ jobs:
# TODO: uncomment after this is tested to limit archiving to PRs to main
# TODO: add extra condition here to check for success of tests??
# Only do it after a successful build and push to main
# if: github.event_name == 'push' && github.ref == 'refs/heads/main'
# if: github.event_name == 'push' && github.ref == 'refs/heads/main'
uses: actions/upload-artifact@v2
with:
name: src
Expand Down
11 changes: 8 additions & 3 deletions .github/workflows/deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,15 @@ jobs:
- name: Checkout repository
uses: actions/checkout@v3

- name: Download artifact
# Necessary since downloading artefacts with an 'overwrite' option is not supported
- name: Empty out src directory
run: rm -rf results/java-lt/src

- name: Download artefact
uses: actions/download-artifact@v3
with:
name: src
path: results/java-lt/

- name: Set dictionary version
# TODO: remove 'test-' prefix!
Expand All @@ -45,5 +50,5 @@ jobs:
GPG_PASSPHRASE: ${{ secrets.GPG_PASSPHRASE }}
SONATYPE_USERNAME: ${{ secrets.SONATYPE_USERNAME }}
SONATYPE_PASSWORD: ${{ secrets.SONATYPE_PASSWORD }}
run: |
mvn clean deploy -P release
working-directory: results/java-lt
run: mvn clean deploy -P release

0 comments on commit 99693f6

Please sign in to comment.