Skip to content

Commit

Permalink
ci: build all rust packages (#9268)
Browse files Browse the repository at this point in the history
### Description

Since we're no longer sharing a repo with turbopack, we can now build
all the crates instead of using cargo groups

### Testing Instructions

<!--
  Give a quick description of steps to test your changes.
-->

---------

Co-authored-by: Chris Olszewski <chris.olszewski@vercel.com>
  • Loading branch information
NicholasLYang and chris-olszewski authored Oct 16, 2024
1 parent f69b43a commit ec80139
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 5 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/turborepo-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -229,7 +229,7 @@ jobs:

- name: Run cargo clippy
run: |
cargo groups clippy turborepo-libraries --features rustls-tls -- --deny clippy::all
cargo clippy --workspace --features rustls-tls -- --deny clippy::all
- name: Run ast-grep lints
run: |
Expand Down Expand Up @@ -260,7 +260,7 @@ jobs:

- name: Run cargo check
run: |
cargo groups check turborepo-libraries
cargo check --workspace
rust_test:
needs: [rust_check]
Expand Down Expand Up @@ -308,8 +308,12 @@ jobs:
run: |
if [ -z "${RUSTC_WRAPPER}" ]; then
unset RUSTC_WRAPPER
fi
cargo groups test turborepo-libraries
fi
if [ "$RUNNER_OS" == "Windows" ]; then
cargo test --workspace --exclude turborepo-napi
else
cargo test --workspace
fi
shell: bash
env:
SCCACHE_BUCKET: turborepo-sccache
Expand Down
2 changes: 1 addition & 1 deletion packages/turbo-repository/rust/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -219,7 +219,7 @@ impl Workspace {
path: info.package_path().to_owned(),
})
.collect::<Vec<WorkspacePackage>>(),
PackageChanges::Some(packages) => packages.into_iter().map(|(p, _)| p).collect(),
PackageChanges::Some(packages) => packages.into_keys().collect(),
};

let mut serializable_packages: Vec<Package> = packages
Expand Down

0 comments on commit ec80139

Please sign in to comment.