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

[TEST] [NO MERGE] add release checklist commenter #34

Closed
wants to merge 1 commit into from

Conversation

amsmith-pro
Copy link

No description provided.

@amsmith-pro amsmith-pro requested a review from a team as a code owner October 17, 2023 20:12
@cla-bot cla-bot bot added the cla-signed label Oct 17, 2023
@amsmith-pro amsmith-pro changed the title feat: add release checklist commenter [TEST] [NO MERGE] add release checklist commenter Oct 17, 2023
@github-actions
Copy link

github-actions bot commented Oct 17, 2023

This PR will generate the following release notes when merged:

Release

✨ Features

  • add release checklist commenter (68b1a45)

@amsmith-pro amsmith-pro force-pushed the am-smith/release-checklist-test branch 3 times, most recently from 285636e to 1deddbf Compare October 17, 2023 20:22
@github-actions
Copy link

github-actions bot commented Oct 17, 2023

📋 Release Checklist ✅

🏡 Check locally with make release-checklist

Release Checklist

📣 Checking log statements...

Test /// Location Type Statement
X X server/build.rs (24) (no_fn_prepend) msg println!("cargo:rerun-if-changed={}", proto_file);
X X server/src/main.rs (47) (no_fn_prepend) msg println!("Starting gRPC server at: {}", full_gr...
X X server/src/main.rs (53) (no_fn_prepend) msg println!("gRPC server running at: {}", full_grp...
X X client-grpc/examples/grpc.rs (33) (no_fn_prepend) msg println!( "NOTE: Ensure the server is ru...
X X client-grpc/examples/grpc.rs (37) (no_fn_prepend) msg println!("Client created");
X X client-grpc/examples/grpc.rs (37) punctuation println!("Client created");
X X client-grpc/examples/grpc.rs (43) (no_fn_prepend) msg println!("RESPONSE={:?}", response.into_inner());

(Some fields hidden. Use -t and -c to show hidden fields.)

💀 Checking #[allow(dead_code)]...

Location Statement
server/src/grpc.rs (25) fn is_ready
client-grpc/src/grpc.rs (24) struct DevopsTestRpcClient

🎀 Checking unwraps and .expect() calls...

Test /// Location
X X server/src/grpc.rs (164)

🚧 Checking TODO items and FIXMEs...

All good! 🎉


🗃 Checking Cargo.toml versions...

Location Statement
server/Cargo.toml (5) version = "0.1.4"
client-grpc/Cargo.toml (5) version = "0.1.4"

⚠️ Comments and test sections not checked by default. Check locally with make release-checklist-full

@amsmith-pro amsmith-pro force-pushed the am-smith/release-checklist-test branch from e085913 to 68b1a45 Compare October 26, 2023 22:04
@github-actions
Copy link

🪲 Unit Testing Coverage 🔎

|| Tested/Total Lines:
|| client-grpc/src/grpc.rs: 0/30
|| server/src/grpc.rs: 0/75
|| server/src/main.rs: 2/21
||
1.59% coverage, 2/126 lines covered

👉 View in Coveralls.io 👈

@amsmith-pro
Copy link
Author

Arrow-air/tf-github#142

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant