qpismont/herald
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases 4 Wiki Activity

1.2: Sandboxing #7

Merged
qpismont merged 17 commits from 1.2 into main 2026-09-20 16:58:36 +02:00
Conversation 18 Commits 17 Files Changed 43
+3 -3
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit ee221b0954 - Show all commits
.woodpecker/tests.yml
+3 -3
View File
@@ -4,19 +4,19 @@ when:
steps: steps:
- name: fmt - name: fmt
qpismont marked this conversation as resolved
Herald commented 2026-09-17 18:21:45 +02:00
Review
Copy Link
Copy Source

Pipeline CI : vérifier que cargo fmt --check, cargo clippy -- -D warnings et cargo test --all-features couvrent bien le nouveau workspace multi-crates (utiliser --workspace), et que les tests du sandbox qui nécessitent Docker/root sont soit exécutés, soit explicitement ignorés avec justification.

Pipeline CI : vérifier que `cargo fmt --check`, `cargo clippy -- -D warnings` et `cargo test --all-features` couvrent bien le nouveau workspace multi-crates (utiliser `--workspace`), et que les tests du sandbox qui nécessitent Docker/root sont soit exécutés, soit explicitement ignorés avec justification.
image: rust:1.97 image: rust:1.98
commands: commands:
- rustup component add rustfmt - rustup component add rustfmt
- cargo fmt --all -- --check - cargo fmt --all -- --check
- name: clippy - name: clippy
image: rust:1.97 image: rust:1.98
commands: commands:
- rustup component add clippy - rustup component add clippy
- cargo clippy --workspace --all-targets --all-features -- -D warnings - cargo clippy --workspace --all-targets --all-features -- -D warnings
- name: test - name: test
image: rust:1.97 image: rust:1.98
commands: commands:
- cargo test --workspace --all-targets - cargo test --workspace --all-targets
1