Skip to content

Conversation

@rical
Copy link
Contributor

@rical rical commented Jul 28, 2025

Description

Checklist

Tick relevant boxes, this PR is-a or has-a:

  • Bugfix
    • Regression tests
    • ChangeLog updates (for next release)
  • Feature
    • YANG model change => revision updated?
    • Regression tests added?
    • ChangeLog updates (for next release)
    • Documentation added?
  • Test changes
    • Checked in changed Readme.adoc (make test-spec)
    • Added new test to group Readme.adoc and yaml file
  • Code style update (formatting, renaming)
  • Refactoring (please detail in commit messages)
  • Build related changes
  • Documentation content changes
    • ChangeLog updated (for major changes)
  • Other (please describe):

@troglobit troglobit marked this pull request as draft July 28, 2025 18:00
rical added 2 commits July 29, 2025 08:03
Signed-off-by: Richard Alpe <richard@bit42.se>
Reduce number of lines. Required = false is the default.

Signed-off-by: Richard Alpe <richard@bit42.se>
@rical rical force-pushed the trigger-github-workflow branch from c445745 to 96dd69e Compare July 29, 2025 07:01
@rical rical changed the title draft: no-merge: .github: noop to trigger github action draft: .github: workflow dispatch fixes Jul 29, 2025
@rical rical force-pushed the trigger-github-workflow branch 4 times, most recently from bf380ba to b865593 Compare July 29, 2025 07:25
Signed-off-by: Richard Alpe <richard@bit42.se>
@rical rical force-pushed the trigger-github-workflow branch from b865593 to fc0a68a Compare July 29, 2025 07:26
@rical rical changed the title draft: .github: workflow dispatch fixes .github: workflow dispatch fixes Jul 29, 2025
@rical rical requested review from mattiaswal and troglobit July 29, 2025 10:11
Copy link
Contributor

@troglobit troglobit left a comment

Choose a reason for hiding this comment

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

Nice and clean, awesome!

@troglobit troglobit marked this pull request as ready for review July 29, 2025 10:28
Copilot AI review requested due to automatic review settings July 29, 2025 10:28
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the GitHub workflow to handle both workflow_dispatch and workflow_call events properly by adding environment variables and moving input definitions to support both trigger types.

  • Adds missing input parameters (target, name) to workflow_dispatch section
  • Creates environment variables to handle inputs from both workflow_dispatch and workflow_call events
  • Updates all job references to use environment variables instead of direct input references

@rical rical merged commit 3d3a4ca into main Jul 29, 2025
12 of 14 checks passed
@rical rical deleted the trigger-github-workflow branch July 29, 2025 11:07
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