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

release: Remove condition on event_name #1675

Merged
merged 1 commit into from
Nov 12, 2024
Merged

Commits on Nov 12, 2024

  1. release: Remove condition on event_name

    This was flawed: even though ci.yaml is called by another workflow, the
    `github` context is associated with the caller workflow¹ so event_name
    is "workflow_dispatch" during a release run.
    
    It doesn't make sense to update the condition to match
    "workflow_dispatch", since there are valid reasons for running ci.yaml
    using workflow_dispatch.
    
    I think it makes more sense to simply remove this half of the condition
    as redundant². The other half of the condition sufficiently ensures that
    the job is run only when called by the release workflow on the default
    branch.
    
    ¹ <https://docs.github.com/en/actions/sharing-automations/reusing-workflows>
    ² <#1634 (comment)>
    victorlin committed Nov 12, 2024
    Configuration menu
    Copy the full SHA
    166628f View commit details
    Browse the repository at this point in the history