Skip to content
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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

evshary
Copy link
Contributor

@evshary evshary commented Dec 12, 2024

Move r2r CI to an independent crate to avoid blocking our release process
Should resolve #359

Copy link

PR missing one of the required labels: {'breaking-change', 'dependencies', 'bug', 'documentation', 'internal', 'enhancement', 'new feature'}

@evshary evshary added the internal Changes not included in the changelog label Dec 12, 2024
@evshary evshary requested a review from JEnoch December 12, 2024 03:46
Signed-off-by: ChenYing Kuo <[email protected]>
Copy link
Contributor Author

@evshary evshary left a 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
Copy link
Contributor Author

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
internal Changes not included in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move ros_test in a separate crate
1 participant