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

build(deps): Bump finch dependencies #346

Merged
merged 1 commit into from
Jun 26, 2024
Merged

Conversation

github-actions[bot]
Copy link
Contributor

Automated changes by create-pull-request GitHub action

Copy link
Member

@austinvazquez austinvazquez left a comment

Choose a reason for hiding this comment

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

Artifact should be basename only. Opened #347 to resolve. Manually edit changes here for correctness.

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: Austin Vazquez <macedonv@amazon.com>
austinvazquez added a commit that referenced this pull request Jun 26, 2024
…347)

Issue #, if available:
Found in #346 

*Description of changes:*
This change updates update-rootfs.sh and update-deps.sh to write the
artifact basename to the ARTIFACT configuration variable.

*Testing done:*
Locally tested in bash:
```
aarch64_deps="aarch64/lima-and-qemu.macos-aarch64.1719307443.tar.gz"
echo "AARCH64_ARTIFACT=$(basename "${aarch64_deps}")"
```

- [x] I've reviewed the guidance in CONTRIBUTING.md

#### License Acceptance

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

Signed-off-by: Austin Vazquez <macedonv@amazon.com>
@austinvazquez
Copy link
Member

Hashes match https://github.com/runfinch/finch-core/actions/runs/9687493542

@austinvazquez austinvazquez merged commit 0e35da9 into main Jun 26, 2024
8 checks passed
@austinvazquez austinvazquez deleted the create-pull-request/patch branch June 26, 2024 23:28
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.

2 participants