Skip to content
Merged
Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
39 commits
Select commit Hold shift + click to select a range
4824f5f
add initial cargo ci
Kasama Oct 13, 2025
c8581e2
merge master into add/cargo-ci
Kasama Oct 15, 2025
1c0406d
update cargo ci
Kasama Oct 24, 2025
c5ea361
[add/cargo-ci]: Merge remote-tracking branch 'origin/master' into add…
bfops Oct 27, 2025
b86cce7
Merge branch 'master' of github.com:clockworklabs/SpacetimeDB into ad…
Kasama Oct 29, 2025
852c13c
fix: PR comments
Kasama Oct 29, 2025
2b2250a
update ci to call cargo for unreal test
Kasama Oct 29, 2025
a9b46b2
update cargo ci docs
Kasama Oct 29, 2025
89b40c0
merge master into add/cargo-ci
Kasama Nov 4, 2025
b318678
Merge branch 'master' of github.com:clockworklabs/SpacetimeDB into ad…
Kasama Nov 6, 2025
fe350f5
fix: temp disable unreal engine tests running from `cargo ci`
Kasama Nov 6, 2025
d546167
Merge branch 'master' of github.com:clockworklabs/SpacetimeDB into ad…
Kasama Nov 6, 2025
ac25ba8
fix cargo lock
Kasama Nov 6, 2025
93e12b7
update smoketests call
Kasama Nov 6, 2025
8bf7367
add directory check for cli-docs
Kasama Nov 6, 2025
a733f12
update cargo ci docs
Kasama Nov 6, 2025
51a0c14
merge master into add/cargo-ci
Kasama Nov 12, 2025
8a37584
Merge branch 'master' of github.com:clockworklabs/SpacetimeDB into ad…
Kasama Nov 12, 2025
44eafca
remove unreal tests from cargo ci initially
Kasama Nov 12, 2025
05d4693
[add/cargo-ci]: Merge remote-tracking branch 'origin/master' into add…
bfops Nov 20, 2025
661af74
[add/cargo-ci]: fixes
bfops Nov 20, 2025
b3e115f
[add/cargo-ci]: review
bfops Nov 20, 2025
e0517e0
[add/cargo-ci]: rm
bfops Nov 20, 2025
8a479f5
[add/cargo-ci]: revert
bfops Nov 20, 2025
dabe11f
[add/cargo-ci]: update self-docs
bfops Nov 20, 2025
f67e95d
[add/cargo-ci]: fix?
bfops Nov 20, 2025
4498429
[add/cargo-ci]: remove stale comment
bfops Nov 20, 2025
a7202ef
[add/cargo-ci]: review
bfops Nov 20, 2025
42ed67c
Merge branch 'master' of github.com:clockworklabs/SpacetimeDB into ad…
Kasama Nov 21, 2025
d13b20a
merge master into add/cargo-ci
Kasama Dec 3, 2025
941763d
workaround: don't use cargo ci for windows jobs
Kasama Dec 3, 2025
7cf2039
merge master into add/cargo-ci
Kasama Dec 8, 2025
7fd39fd
add: bash macro for python3 check
Kasama Dec 8, 2025
922dc0c
Merge branch 'master' into add/cargo-ci
bfops Dec 9, 2025
4070974
Apply suggestion from @bfops
bfops Dec 9, 2025
3fb016d
Apply suggestion from @bfops
bfops Dec 9, 2025
a3a2ea9
Merge branch 'master' of github.com:clockworklabs/SpacetimeDB into ad…
Kasama Dec 10, 2025
77a71ef
Merge branch 'add/cargo-ci' of github.com:clockworklabs/SpacetimeDB i…
Kasama Dec 10, 2025
94302a8
Update tools/ci/src/main.rs
Kasama Dec 10, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .cargo/config.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ rustflags = ["--cfg", "tokio_unstable"]

[alias]
bump-versions = "run -p upgrade-version --"
ci = "run -p ci --"

[target.x86_64-pc-windows-msvc]
# Use a different linker. Otherwise, the build fails with some obscure linker error that
Expand Down
142 changes: 58 additions & 84 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
include:
- { runner: spacetimedb-runner, smoketest_args: --docker }
- { runner: windows-latest, smoketest_args: --no-build-cli }
runner: [ spacetimedb-runner, windows-latest ]
Comment thread
bfops marked this conversation as resolved.
Outdated
runner: [spacetimedb-runner, windows-latest]
runs-on: ${{ matrix.runner }}
steps:
- name: Find Git ref
Expand Down Expand Up @@ -70,8 +70,8 @@
- name: Install psycopg2
run: python -m pip install psycopg2-binary
- name: Run smoketests
Comment thread
bfops marked this conversation as resolved.
Outdated
# Note: clear_database and replication only work in private
run: python -m smoketests ${{ matrix.smoketest_args }} -x clear_database replication
run: cargo ci smoketests -- ${{ matrix.smoketest_args }}
Comment thread
bfops marked this conversation as resolved.
Outdated

- name: Stop containers (Linux)
if: always() && runner.os == 'Linux'
run: docker compose down
Expand Down Expand Up @@ -103,26 +103,9 @@
with:
global-json-file: global.json

- name: Create /stdb dir
run: |
sudo mkdir /stdb
sudo chmod 777 /stdb

- name: Run cargo test
#Note: Unreal tests will be run separately
run: cargo test --all -- --skip unreal

- name: Check that the test outputs are up-to-date
run: bash tools/check-diff.sh

- name: Ensure C# autogen bindings are up-to-date
run: |
cargo run -p spacetimedb-codegen --example regen-csharp-moduledef
bash tools/check-diff.sh crates/bindings-csharp

- name: C# bindings tests
working-directory: crates/bindings-csharp
run: dotnet test -warnaserror
run: cargo ci test

lints:
name: Lints
Expand All @@ -138,30 +121,8 @@
with:
global-json-file: global.json

- name: Run cargo fmt
run: cargo fmt --all -- --check

- name: Run cargo clippy
run: cargo clippy --all --tests --benches -- -D warnings

- name: Run C# formatting check
working-directory: crates/bindings-csharp
run: |
dotnet tool restore
dotnet csharpier --check .

- name: Run `cargo doc` for bindings crate
# `bindings` is the only crate we care strongly about documenting,
# since we link to its docs.rs from our website.
# We won't pass `--no-deps`, though,
# since we want everything reachable through it to also work.
# This includes `sats` and `lib`.
working-directory: crates/bindings
env:
# Make `cargo doc` exit with error on warnings, most notably broken links
RUSTDOCFLAGS: '--deny warnings'
run: |
cargo doc
- name: Run ci lint
run: cargo ci lint

wasm_bindings:
name: Build and test wasm bindings
Expand All @@ -173,20 +134,7 @@
- run: echo ::add-matcher::.github/workflows/rust_matcher.json

- name: Run bindgen tests
run: cargo test -p spacetimedb-codegen

# Make sure the `Cargo.lock` file reflects the latest available versions.
# This is what users would end up with on a fresh module, so we want to
# catch any compile errors arising from a different transitive closure
# of dependencies than what is in the workspace lock file.
#
# For context see also: https://github.com/clockworklabs/SpacetimeDB/pull/2714
- name: Update dependencies
run: cargo update

- name: Build module-test
run: cargo run -p spacetimedb-cli -- build --project-path modules/module-test

run: cargo ci wasm-bindings

publish_checks:
name: Check that packages are publishable
Expand Down Expand Up @@ -239,19 +187,9 @@
run: sudo apt install -y libssl-dev

- name: Build spacetimedb-update
Comment thread
Kasama marked this conversation as resolved.
run: cargo build --features github-token-auth --target ${{ matrix.target }} -p spacetimedb-update

- name: Run self-install
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
shell: bash
run: |
ROOT_DIR="$(mktemp -d)"
# NOTE(bfops): We need the `github-token-auth` feature because we otherwise tend to get ratelimited when we try to fetch `/releases/latest`.
# My best guess is that, on the GitHub runners, the "anonymous" ratelimit is shared by *all* users of that runner (I think this because it
# happens very frequently on the `macos-runner`, but we haven't seen it on any others).
cargo run --features github-token-auth --target ${{ matrix.target }} -p spacetimedb-update -- self-install --root-dir="${ROOT_DIR}" --yes
"${ROOT_DIR}"/spacetime --root-dir="${ROOT_DIR}" help
run: cargo ci update-flow --target=${{ matrix.target }}

unreal_engine_tests:
name: Unreal Engine Tests
Expand All @@ -271,25 +209,34 @@
# without this (reassigning env vars and stuff), but was unable to get it to work and it felt like an uphill battle.
options: --user 0:0
steps:
# Uncomment this before merging so that it will run properly if run manually through the GH actions flow. It was playing weird with rolled back
# commits though.
# - name: Find Git ref
# env:
# GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# shell: bash
# run: |
# PR_NUMBER="${{ github.event.inputs.pr_number || null }}"
# if test -n "${PR_NUMBER}"; then
# GIT_REF="$( gh pr view --repo clockworklabs/SpacetimeDB $PR_NUMBER --json headRefName --jq .headRefName )"
# else
# GIT_REF="${{ github.ref }}"
# fi
# echo "GIT_REF=${GIT_REF}" >>"$GITHUB_ENV"
# Uncomment this before merging so that it will run properly if run manually through the GH actions flow. It was playing weird with rolled back
# commits though.
# - name: Find Git ref
# env:
# GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
# shell: bash
# run: |
# PR_NUMBER="${{ github.event.inputs.pr_number || null }}"
# if test -n "${PR_NUMBER}"; then
# GIT_REF="$( gh pr view --repo clockworklabs/SpacetimeDB $PR_NUMBER --json headRefName --jq .headRefName )"
# else
# GIT_REF="${{ github.ref }}"
# fi
# echo "GIT_REF=${GIT_REF}" >>"$GITHUB_ENV"
- name: Checkout sources
uses: actions/checkout@v4
with:
ref: ${{ env.GIT_REF }}
- uses: dsherret/rust-toolchain-file@v1
- name: Install unreal engine test dependencies
working-directory: sdks/unreal
env:
UE_ROOT_PATH: /home/ue4/UnrealEngine
run: |

apt-get update
apt-get install -y acl curl ca-certificates
Comment thread
bfops marked this conversation as resolved.

- name: Run Unreal Engine tests
working-directory: sdks/unreal
env:
Expand Down Expand Up @@ -337,7 +284,34 @@
cargo test
'

ci_command_docs:
name: Check CI command docs
runs-on: ubuntu-latest
steps:
- name: Find Git ref
env:
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
shell: bash
run: |
PR_NUMBER="${{ github.event.inputs.pr_number || null }}"
if test -n "${PR_NUMBER}"; then
GIT_REF="$( gh pr view --repo clockworklabs/SpacetimeDB $PR_NUMBER --json headRefName --jq .headRefName )"
else
GIT_REF="${{ github.ref }}"
fi
echo "GIT_REF=${GIT_REF}" >>"$GITHUB_ENV"

- name: Checkout sources
uses: actions/checkout@v4
with:
ref: ${{ env.GIT_REF }}

- uses: dsherret/rust-toolchain-file@v1

- name: Check for docs change
run: cargo ci self-docs --check

cli_docs:

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {contents: read}
name: Check CLI docs
Comment thread
bfops marked this conversation as resolved.
permissions: read-all
runs-on: ubuntu-latest
Expand Down
Loading