Skip to content

Commit e137630

Browse files
authored
ci: More CI checks in cubesql and cubejs-backend-native (#8595)
* Fixes in large_model.rs after refactorings * Setup cargo check in CI for all targets in cubesql * Run rustfmt in all cubesql workspace in CI * Make clippy locked and check all targets in CI for cubejs-backend-native
1 parent 756f659 commit e137630

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

.github/workflows/rust-cubesql.yml

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,13 +40,16 @@ jobs:
4040
key: cubesql-${{ runner.OS }}-x86_64-unknown-linux-gnu
4141
shared-key: cubesql-${{ runner.OS }}-x86_64-unknown-linux-gnu
4242
- name: Lint CubeSQL
43-
run: cd rust/cubesql/cubesql && cargo fmt --all -- --check
43+
run: cd rust/cubesql && cargo fmt --all -- --check
4444
- name: Lint Native
4545
run: cd packages/cubejs-backend-native && cargo fmt --all -- --check
46+
# TODO replace with clippy once cubesql is ready
47+
- name: Check CubeSQL
48+
run: cd rust/cubesql && cargo check --locked --workspace --all-targets --keep-going
4649
- name: Clippy Native
47-
run: cd packages/cubejs-backend-native && cargo clippy -- -D warnings
50+
run: cd packages/cubejs-backend-native && cargo clippy --locked --workspace --all-targets --keep-going -- -D warnings
4851
- name: Clippy Native (with Python)
49-
run: cd packages/cubejs-backend-native && cargo clippy --features python -- -D warnings
52+
run: cd packages/cubejs-backend-native && cargo clippy --locked --workspace --all-targets --keep-going --features python -- -D warnings
5053
# CubeSQL is not ready for Clippy
5154
#- name: Clippy CubeSQL
5255
# run: cd rust/cubesql && cargo clippy -- -D warnings

rust/cubesql/cubesql/benches/large_model.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ use uuid::Uuid;
2020
macro_rules! bench_large_model {
2121
($DIMS:expr, $NAME:expr, $QUERY_FN:expr, $CRITERION:expr) => {{
2222
let context = Arc::new(
23-
futures::executor::block_on(create_test_cube_context(
24-
create_test_postgresql_cube_context($DIMS),
23+
futures::executor::block_on(create_test_postgresql_cube_context(
24+
get_large_model_test_tenant_ctx($DIMS),
2525
))
2626
.unwrap(),
2727
);

0 commit comments

Comments
 (0)