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.

Sign up for GitHub

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

Jump to bottom

Output image digest #101

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Output image digest #101

merged 1 commit into from
Oct 17, 2024

Conversation

timdittler
Copy link
Contributor

Type of Change

  • Bugfix
  • Enhancement / new feature
  • Refactoring
  • Documentation

Description

  • Actions don't have access to vars and secrets. Therefore this approach didn't work.
  • Instead I will now output the digest and use it in the workflow
  • Document output that was already there

Checklist

  • Write tests
  • Make sure all tests pass
  • Update documentation
  • Review the Contributing Guideline and sign CLA
  • Reference relevant issue(s) and close them after merging

@timdittler timdittler requested a review from a team as a code owner October 17, 2024 14:03
@timdittler timdittler mentioned this pull request Oct 17, 2024
Merged
9 tasks
@timdittler timdittler merged commit b0cb186 into main Oct 17, 2024
6 checks passed
@timdittler timdittler deleted the upwind2 branch October 17, 2024 14:25
@github-actions github-actions bot locked and limited conversation to collaborators Oct 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@0x46616c6b 0x46616c6b 0x46616c6b approved these changes

@monotek monotek Awaiting requested review from monotek monotek is a code owner automatically assigned from Staffbase/diablo

@veerendra2 veerendra2 Awaiting requested review from veerendra2 veerendra2 is a code owner automatically assigned from Staffbase/diablo

@kaitimmer kaitimmer Awaiting requested review from kaitimmer kaitimmer is a code owner automatically assigned from Staffbase/diablo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@timdittler @0x46616c6b