Skip to content

pr commenting: use native github CLI#432

Merged
rugeli merged 2 commits intomainfrom
feature/github-cli
Nov 20, 2025
Merged

pr commenting: use native github CLI#432
rugeli merged 2 commits intomainfrom
feature/github-cli

Conversation

@rugeli
Copy link
Collaborator

@rugeli rugeli commented Nov 17, 2025

Problem

closes #422
documentation: https://cli.github.com/manual/gh_pr_comment

Solution

  • removed iterative/setup-cml@v2
  • updated both comment steps to use native gh cli commands
    • now uses an html comment marker to identify workflow-generated comments
  • added logic: detect and delete if an existing comment found, post a fresh comment containing the latest report

Type of change

  • New feature (non-breaking change which adds functionality)

Steps to Verify:

  1. everything in the Comment job should behave the same as before
  2. verify the comment is correctly post, updated, and only one exists at a time in this pr

@github-actions
Copy link
Contributor

Packing analysis report

Analysis for packing results located at cellpack/tests/outputs/test_spheres/spheresSST

Ingredient name Encapsulating radius Average number packed
ext_A 25 236.0

Packing image

Packing image

Distance analysis

Expected minimum distance: 50.00
Actual minimum distance: 50.01

Ingredient key Pairwise distance distribution
ext_A Distance distribution ext_A

@rugeli rugeli changed the title Feature/GitHub cli pr commenting: use native github CLI Nov 17, 2025
@rugeli rugeli requested review from meganrm and mogres November 17, 2025 19:53
Copy link
Collaborator

@mogres mogres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great! Thanks for the update!

@rugeli rugeli merged commit 332d07d into main Nov 20, 2025
10 checks passed
@rugeli rugeli deleted the feature/github-cli branch November 20, 2025 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use github CLI for PR comments and remove iterative/setup-cml action

3 participants