diff --git a/.github/workflows/post-merge.yml b/.github/workflows/post-merge.yml index de31c28cc90e0..94553608a2f48 100644 --- a/.github/workflows/post-merge.yml +++ b/.github/workflows/post-merge.yml @@ -35,13 +35,12 @@ jobs: cd src/ci/citool - printf "*This is an experimental post-merge analysis report. You can ignore it.*\n\n" > output.log - printf "<details>\n<summary>Post-merge report</summary>\n\n" >> output.log + printf "<details>\n<summary>What is this?</summary>\n" >> output.log + printf "This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.\n" >> output.log + printf "</details>\n\n" >> output.log cargo run --release post-merge-report ${PARENT_COMMIT} ${{ github.sha }} >> output.log - printf "</details>\n" >> output.log - cat output.log gh pr comment ${HEAD_PR} -F output.log diff --git a/src/ci/citool/src/main.rs b/src/ci/citool/src/main.rs index 9e4b558d77aa0..5a84ecb5e47e5 100644 --- a/src/ci/citool/src/main.rs +++ b/src/ci/citool/src/main.rs @@ -20,7 +20,7 @@ use crate::cpu_usage::load_cpu_usage; use crate::datadog::upload_datadog_metric; use crate::jobs::RunType; use crate::metrics::{JobMetrics, download_auto_job_metrics, download_job_metrics, load_metrics}; -use crate::utils::{load_env_var, output_details}; +use crate::utils::load_env_var; const CI_DIRECTORY: &str = concat!(env!("CARGO_MANIFEST_DIR"), "/.."); const DOCKER_DIRECTORY: &str = concat!(env!("CARGO_MANIFEST_DIR"), "/../docker"); @@ -175,13 +175,6 @@ fn postprocess_metrics( fn post_merge_report(db: JobDatabase, current: String, parent: String) -> anyhow::Result<()> { let metrics = download_auto_job_metrics(&db, &parent, ¤t)?; - output_details("What is this?", || { - println!( - r#"This is an experimental post-merge analysis report that shows differences in -test outcomes between the merged PR and its parent PR."# - ); - }); - println!("\nComparing {parent} (parent) -> {current} (this PR)\n"); output_test_diffs(metrics);