Skip to content

Commit

Permalink
fix: πŸ› naming consistency
Browse files Browse the repository at this point in the history
βœ… Closes: #5
  • Loading branch information
jlarmstrongiv committed May 13, 2021
1 parent 645e166 commit 36e39a2
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 4 deletions.
1 change: 1 addition & 0 deletions .github/actions/compile/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import * as core from '@actions/core';
import main from './main';

try {
// keep format for backwards compatibility nodejs10.x-tf1.7.4.br
const lambda = `nodejs${core.getInput('lambda')}.x`;
const tensorflow = core.getInput('tensorflow');
const binary = core.getInput('binary');
Expand Down
11 changes: 7 additions & 4 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,8 @@ jobs:
with:
lambda: ${{ matrix.lambda }}
tensorflow: ${{ matrix.tensorflow }}
binary: '${{ matrix.lambda }}-tf${{ matrix.tensorflow }}.br'
# keep format for backwards compatibility nodejs10.x-tf1.7.4.br
binary: 'nodejs${{ matrix.lambda }}.x-tf${{ matrix.tensorflow }}.br'

- name: List files
run: |
Expand All @@ -71,8 +72,9 @@ jobs:
uses: actions/upload-artifact@v2
with:
if-no-files-found: error
name: '${{ matrix.lambda }}-tf${{ matrix.tensorflow }}'
path: './${{ matrix.lambda }}-tf${{ matrix.tensorflow }}.br'
# keep format for backwards compatibility nodejs10.x-tf1.7.4.br
name: 'nodejs${{ matrix.lambda }}.x-tf${{ matrix.tensorflow }}'
path: './nodejs${{ matrix.lambda }}.x-tf${{ matrix.tensorflow }}.br'
retention-days: 2

publish:
Expand All @@ -97,8 +99,9 @@ jobs:
- name: Add matrix.json
working-directory: ./tfjs-node-lambda-releases
# https://stackoverflow.com/a/35589472
# keep format for backwards compatibility nodejs10.x-tf1.7.4.br
run: |
echo '{"lambda":${{ needs.matrix.outputs.lambda }},"tensorflow":${{ needs.matrix.outputs.tensorflow }}}' >| matrix.json
echo '{"lambda":'$(node --print 'JSON.stringify(${{ needs.matrix.outputs.lambda }}.map(version => `nodejs${version}.x`))')',"tensorflow":${{ needs.matrix.outputs.tensorflow }}}' >| matrix.json
- name: Fix folder structure
working-directory: ./tfjs-node-lambda-releases
Expand Down

0 comments on commit 36e39a2

Please sign in to comment.