Skip to content

ci: support optional pr-number and commit-sha in reusable governance + run on draft PRs#34

Open
damaz91 wants to merge 2 commits into
Universal-Commerce-Protocol:mainfrom
damaz91:update-reusable-governance-inputs
Open

ci: support optional pr-number and commit-sha in reusable governance + run on draft PRs#34
damaz91 wants to merge 2 commits into
Universal-Commerce-Protocol:mainfrom
damaz91:update-reusable-governance-inputs

Conversation

@damaz91

@damaz91 damaz91 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Description

This PR updates the reusable-governance.yml workflow to accept optional pr-number and commit-sha inputs. If these inputs are not provided, it falls back to the default pull_request context.

This allows the workflow to be triggered from other events (like workflow_run or repository_dispatch) where the pull_request context might not be directly available, but the PR number and commit SHA are known.

It also allow the governance scripts to run for draft PRs, instead of skipping it.

Category (Required)

Please select one or more categories that apply to this change.

  • Core Protocol: Changes to the base communication layer, global context, or breaking refactors. (Requires Technical Council approval)
  • Governance/Contributing: Updates to GOVERNANCE.md, CONTRIBUTING.md, or CODEOWNERS. (Requires Governance Council approval)
  • Capability: New schemas (Discovery, Cart, etc.) or extensions. (Requires Maintainer approval)
  • Documentation: Updates to README, or documentations regarding schema or capabilities. (Requires Maintainer approval)
  • Infrastructure: CI/CD, Linters, or build scripts. (Requires DevOps Maintainer approval)
  • Maintenance: Version bumps, lockfile updates, or minor bug fixes. (Requires DevOps Maintainer approval)
  • SDK: Language-specific SDK updates and releases. (Requires DevOps Maintainer approval)
  • Samples / Conformance: Maintaining samples and the conformance suite. (Requires Maintainer approval)
  • UCP Schema: Changes to the ucp-schema tool (resolver, linter, validator). (Requires Maintainer approval)
  • Community Health (.github): Updates to templates, workflows, or org-level configs. (Requires DevOps Maintainer approval)

Related Issues

None.

Checklist

  • I have followed the Contributing Guide (including Conventional Commits title requirements and ! for breaking changes).
  • I have updated the documentation (if applicable).
  • My changes pass all local linting and formatting checks.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • (For Core/Capability) I have included/updated the relevant JSON schemas.
  • I have regenerated Python Pydantic models by running generate_models.sh under python_sdk.

@damaz91 damaz91 requested a review from a team as a code owner June 30, 2026 09:55
@damaz91 damaz91 requested review from aksbro-gpu and ptiper June 30, 2026 09:55
Remove condition to run job only for non-draft PRs.
@damaz91 damaz91 changed the title ci: support optional pr-number and commit-sha in reusable governance ci: support optional pr-number and commit-sha in reusable governance + run on draft PRs Jun 30, 2026
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.

1 participant