Skip to content

(fix): new keep alive workflow#16

Merged
ilan-gold merged 4 commits into
mainfrom
ig/new_keepalive
May 5, 2025
Merged

(fix): new keep alive workflow#16
ilan-gold merged 4 commits into
mainfrom
ig/new_keepalive

Conversation

@ilan-gold
Copy link
Copy Markdown
Contributor

No description provided.

@ilan-gold ilan-gold requested review from flying-sheep and grst May 5, 2025 07:36
steps:
- uses: actions/checkout@v4
- uses: gautamkrishnar/keepalive-workflow@v2
- name: Re-enable workflow
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think you still need a checkout or hardcode the name of the yml file

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Could you explain why? I don't see anything that would necessitate it so I'd like to understand what I'm missing.

Copy link
Copy Markdown
Contributor

@grst grst May 5, 2025

Choose a reason for hiding this comment

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

the original action additionally has

case "${GITHUB_WORKFLOW_REF:?}" in
          "${GITHUB_REPOSITORY:?}"/.github/workflows/*.y*ml@*) ;;
          *) false ;;
        esac

which somehow globs for the yml files. Now that I look at it again, I'm not sure it's required.

In any case ${workflow} in the API call should be the name of the workflow yml file. But I'm not really sure what the expression ${GITHUB_WORKFLOW_REF%%@*} evaluates to.

Comment thread .github/workflows/integration-test.yml Outdated
Comment thread .github/workflows/integration-test.yml Outdated
Co-authored-by: Gregor Sturm <mail@gregor-sturm.de>
@ilan-gold ilan-gold merged commit 3e717f3 into main May 5, 2025
11 of 15 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.

3 participants