-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move CI to an independent crate. #362
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: ChenYing Kuo <[email protected]>
PR missing one of the required labels: {'breaking-change', 'dependencies', 'bug', 'documentation', 'internal', 'enhancement', 'new feature'} |
Signed-off-by: ChenYing Kuo <[email protected]>
Signed-off-by: ChenYing Kuo <[email protected]>
Signed-off-by: ChenYing Kuo <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've tested with cargo clippy --all-targets --all-features -- --deny warnings
and it works fine.
@@ -46,7 +46,7 @@ jobs: | |||
- name: Code format check | |||
run: cargo fmt --check -- --config "unstable_features=true,imports_granularity=Crate,group_imports=StdExternalCrate" | |||
- name: Clippy | |||
run: cargo clippy --all --examples -- -D warnings | |||
run: cargo clippy --workspace --all-targets --all-features -- --deny warnings |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JEnoch I updated the clippy argument here to match the one we used in Zenoh. I added --all-targets
and --all-features
to avoid the possible changes in the future. Also, --all
is replaced by --workspace
.
Move
r2r
CI to an independent crate to avoid blocking our release processShould resolve #359