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

Update workflow to access aws for dependabot #281

Merged
merged 4 commits into from
Aug 22, 2024
Merged

Conversation

rugeli
Copy link
Collaborator

@rugeli rugeli commented Aug 6, 2024

Problem

What is the problem this work solves, including
the analyze workflow fails to run for dependabot PRs due to credential errors, example

Solution

What I/we did to solve this problem

  • added DEPENDABOT_AWS_ACCESS_KEY_ID and DEPENDABOT_AWS_SECRET_ACCESS_KEY to dependabot secrets
  • updated analyze workflow to handle aws access for dependabot PRs

Note: I tested the workflow syntax on this branch but wasn't able to test dependabot PRs until we merge the changes to main and re-run the open dependabot pr for further testing.

Type of change

Please delete options that are not relevant.

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

@rugeli rugeli requested review from meganrm and mogres August 6, 2024 23:37
Copy link

github-actions bot commented Aug 6, 2024

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 merged commit 0b756e4 into main Aug 22, 2024
7 checks passed
@rugeli rugeli deleted the feature/dependabot-creds branch August 22, 2024 18:25
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.

3 participants