Skip to content

[CI] DRY fix action names in PR-action workflow (#5894)#8913

Open
pratik-mahalle wants to merge 1 commit intoopen-telemetry:mainfrom
pratik-mahalle:ci-dry-fix-action-names
Open

[CI] DRY fix action names in PR-action workflow (#5894)#8913
pratik-mahalle wants to merge 1 commit intoopen-telemetry:mainfrom
pratik-mahalle:ci-dry-fix-action-names

Conversation

@pratik-mahalle
Copy link
Copy Markdown
Contributor

@pratik-mahalle pratik-mahalle commented Jan 15, 2026

This PR keeps the PR-action workflow DRY by dynamically validating fix action names against npm run -s _list:fix:* instead of relying on hardcoded patterns.

Benefits

  • Prevents script-name mismatches like this one
  • No need to manually maintain a list of valid fix actions in the workflow
  • Invalid /fix:xxx commands now fail fast with a clear error message showing valid options

Fixes #5894

Signed-off-by: Pratik Mahalle <mahallepratik683@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/infra Repo CI & infrastructure

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[CI] Avoid repeating fix action names in PR-action workflow

2 participants