Skip to content

Conversation

sidpalas
Copy link
Contributor

@sidpalas sidpalas commented Oct 2, 2025

  • add support for an optional digger-version input which enables pinning usage of the digger action to the commit hash without incurring the performance penalty of needing to rebuild from source.
  • extract binary mode (local/prebuilt/build) into named variable to clarify conditionals

shell: bash
if: inputs.setup-google-cloud == 'true'
- uses: actions/checkout@v4
- uses: actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955 # v4.3.0
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a drive by fix for one action I missed in #2277

if: ${{ inputs.local-dev-mode == 'true' }}
env:
actionref: ${{ github.action_ref }}
DIGGER_VERSION: ${{ github.action_ref }}
Copy link
Contributor Author

@sidpalas sidpalas Oct 2, 2025

Choose a reason for hiding this comment

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

Updated here for consistency, but I don't actually think this gets used...

@sidpalas sidpalas changed the title feat(run-digger-cli): add digger-version input feat(run-digger-action): add digger-version input Oct 2, 2025
@motatoes motatoes marked this pull request as ready for review October 2, 2025 23:34
@sidpalas
Copy link
Contributor Author

sidpalas commented Oct 7, 2025

@motatoes -- anything else you think needs to be done here before it is ready to merge?

@motatoes
Copy link
Contributor

motatoes commented Oct 7, 2025

Thanks @sidpalas ! merging 🥳

@motatoes motatoes merged commit 869b24d into diggerhq:develop Oct 7, 2025
7 checks passed
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