Skip to content

Commit

Permalink
Fix web workflow
Browse files Browse the repository at this point in the history
Remove the env variable from deno deploy and unify the linux and macos build step
  • Loading branch information
SchahinRohani committed Sep 27, 2024
1 parent 9600839 commit 97d6254
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 12 deletions.
12 changes: 2 additions & 10 deletions .github/workflows/web.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -69,29 +69,21 @@ jobs:
uses: >- # v4
DeterminateSystems/magic-nix-cache-action@fc6aaceb40b9845a02b91e059ec147e78d1b4e41
- name: Test Build on Ubuntu
if: matrix.os == 'ubuntu-24.04' && github.event_name == 'pull_request'
- name: Test Build
if: github.event_name == 'pull_request'
working-directory: web/platform
run: |
nix develop --impure --command bash -c "
bun setup && bun docs && bun run build
"
- name: Test Build on macOS
if: matrix.os == 'macos-14' && github.event_name == 'pull_request'
working-directory: web/platform
run: |
nix develop --impure --command bash -c "bun setup && bun run build"
- name: Production deployment
if: matrix.os == 'ubuntu-24.04' && github.ref == 'refs/heads/main'
working-directory: web/platform
env:
DENO_DEPLOY_TOKEN: ${{ secrets.DENO_DEPLOY_TOKEN }}
RESEND_API_KEY: ${{ secrets.RESEND_API_KEY }}
run: |
nix develop --impure --command bash -c "
bun prod --project=nativelink --org=nativelink \
--token=$DENO_DEPLOY_TOKEN \
--env=RESEND_API_KEY=$RESEND_API_KEY
"
4 changes: 2 additions & 2 deletions web/platform/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,10 +30,10 @@ This project requires `bun` and `deno`. The nix flake ships compatible versions.
bun install

# Rebuild the docs reference. (Note MacOS: Use Bun & Bazel from Host)
bun run docs.build
bun run build.docs

# Generate the simple parts of the autogenerated docs.
bun run docs.generate
bun run generate.docs

# Rebuild everything. Make sure to remove the `dist` directory beforehand.
bun run build
Expand Down

0 comments on commit 97d6254

Please sign in to comment.