[build] Combine Rust build and test CI jobs

[titusfortner]

💥 What does this PR do?

Combines the separate build job into the tests job in the Rust CI workflow (.github/workflows/ci-rust.yml).

Previously there was a standalone build job running bazel build //rust:selenium-manager, with the tests matrix gated on it via needs: build.
Since bazel test //rust/... already compiles //rust:selenium-manager (and the rest of the tree) when producing the test binaries, the dedicated build job was redundant. It has been removed and tests now runs directly.

This should save ~3-4 minutes wall clock on runs.

🔧 Implementation Notes

The old build job provided a fast, single-platform (ubuntu) "does it compile" gate before fanning out to the 3-OS test matrix. With it removed, a compile error now surfaces from all three matrix legs in parallel instead of gating on one cheap build first — functionally equivalent, with slightly more runner usage on a broken build. Downstream platform jobs still needs: tests, so the overall ordering is preserved.

🤖 AI assistance

• No substantial AI assistance used
• AI assisted (complete below)
  • Tool(s): Claude Code
  • What was generated:
  • I reviewed all AI output and can explain the change

💡 Additional Considerations

🔄 Types of changes

• Cleanup (formatting, renaming)

[qodo-code-review]

Review Summary by Qodo

Remove redundant Rust build job from CI workflow

✨ Enhancement

Walkthroughs

Description

• Removes redundant standalone build job from CI workflow
• Eliminates unnecessary compilation gate before test matrix
• Reduces CI runtime by ~3-4 minutes per workflow run
• Tests job now runs directly without build dependency

Diagram

flowchart LR
  A["CI Workflow"] --> B["Tests Job<br/>3-OS Matrix"]
  B --> C["Compile & Test<br/>All Platforms"]
  style A fill:#e1f5ff
  style B fill:#c8e6c9
  style C fill:#fff9c4

Loading

File Changes

1. .github/workflows/ci-rust.yml ⚙️ Configuration changes +0/-8

Eliminate redundant build job from Rust CI

• Removed standalone build job that ran bazel build //rust:selenium-manager
• Removed needs: build dependency from tests job
• Tests job now runs directly without build gate
• Compilation still occurs during bazel test //rust/... execution

.github/workflows/ci-rust.yml

---

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Previous review results

Review updated until commit a98e0ea

Results up to commit cb313c8

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

[qodo-code-review]

Code review by qodo was updated up to the latest commit a98e0ea