Skip to content

Use native gh CLI for requirements actions#453

Open
mogres wants to merge 2 commits intomainfrom
admin/native-gh-cli
Open

Use native gh CLI for requirements actions#453
mogres wants to merge 2 commits intomainfrom
admin/native-gh-cli

Conversation

@mogres
Copy link
Collaborator

@mogres mogres commented Mar 5, 2026

  • update UV setup action to version 7
  • update Dependabot configuration

Problem

closes #452

Solution

Used native gh CLI implementation to create pull requests instead of a third party action

with @rugeli

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Change summary:

Steps to Verify:

Actions should run as expected

* update UV setup to version 7
* update Dependabot configuration
@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

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

@mogres mogres marked this pull request as ready for review March 5, 2026 17:50
@mogres mogres requested review from Copilot and rugeli March 5, 2026 17:50
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@rugeli rugeli force-pushed the admin/native-gh-cli branch from a0df65f to 3d6b2fd Compare March 5, 2026 18:45
Copy link
Collaborator

@rugeli rugeli left a comment

Choose a reason for hiding this comment

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

Thanks for the quick workaround on this! I did some testing(sorry for the force-push noise on this branch and the pr I just closed)

just one minor comment, otherwise it works great!

Comment on lines +67 to +69
permissions:
contents: write
pull-requests: write
Copy link
Collaborator

Choose a reason for hiding this comment

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

we can safely remove this block now since the permissions are defined when the token(WORKFLOW_PAT) is created

@rugeli rugeli requested a review from Copilot March 5, 2026 19:36
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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 native gh CLI for actions and workflows

3 participants