Skip to content

Commit

Permalink
Merge pull request #3544 from ncave/rust
Browse files Browse the repository at this point in the history
Separate CI job for standalone
  • Loading branch information
ncave authored Oct 15, 2023
2 parents 8a20457 + 5791848 commit 597fd2c
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 3 deletions.
18 changes: 18 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,24 @@ jobs:
- name: Fable Tests
run: ./build.sh test integration

# Separate build job for Standalone
build-standalone:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v2

- name: Setup .NET
uses: actions/setup-dotnet@v1
with:
dotnet-version: '6.0.x'

- name: Setup Node.js environment
uses: actions/setup-node@v2.4.1

- name: Fable Tests
run: ./build.sh test standalone

# Separate build job for Python since we use a test matrix (will run in parallell)
build-python:
runs-on: ubuntu-latest
Expand Down
7 changes: 4 additions & 3 deletions src/Fable.Build/Test/JavaScript.fs
Original file line number Diff line number Diff line change
Expand Up @@ -105,10 +105,11 @@ let private handleMainTests (isWatch: bool) (noDotnet: bool) =

testReact false

let isCI = Environment.GetEnvironmentVariable("CI") |> Option.ofObj
// let isCI = Environment.GetEnvironmentVariable("CI") |> Option.ofObj

if isCI.IsSome then
Standalone.handleStandaloneFast ()
// standalone will be tested by a separate CI job
// if isCI.IsSome then
// Standalone.handleStandaloneFast ()

let handle (args: string list) =
let isReactOnly = args |> List.contains "--react-only"
Expand Down
1 change: 1 addition & 0 deletions src/Fable.Build/Test/Standalone.fs
Original file line number Diff line number Diff line change
Expand Up @@ -47,4 +47,5 @@ let handleStandaloneFast () =
Command.Run("npx", mochaCommand, workingDirectory = standaloneBuildDest)

let handle (args: string list) =
BuildFableLibraryJavaScript().Run()
handleStandaloneFast ()

0 comments on commit 597fd2c

Please sign in to comment.