Skip to content

Test PRTK against invalid branch, third attempt#106

Closed
reuterbal wants to merge 3 commits into
ecmwf-ifs:nabr-cy50r1-configfrom
reuterbal:nabr-test-prtk
Closed

Test PRTK against invalid branch, third attempt#106
reuterbal wants to merge 3 commits into
ecmwf-ifs:nabr-cy50r1-configfrom
reuterbal:nabr-test-prtk

Conversation

@reuterbal

Copy link
Copy Markdown
Collaborator

Description

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

msleigh and others added 3 commits May 26, 2026 14:43
Replace the standalone prtk assign-reviewers step with prtk check, the
recommended repository automation entry point in the current version of
prtk. Add a checks: section to MAINTAINERS.yml enabling ensure-assignee,
validate-target-branch (master, develop), assign-reviewers, and
copyright-audit. Expand the workflow event types, add issues: write
permission and a concurrency group, and pass the default branch and
event metadata to prtk check.

Also gate the job with `if: github.repository_owner == 'ecmwf-ifs'`
so PR automation does not run on personal forks.
@github-actions

Copy link
Copy Markdown

@reuterbal: There is 1 issue to fix before this PR can proceed:

  1. Validate target branch
    Target branch 'nabr-cy50r1-config' is not allowed
    Change the PR base branch in GitHub to one of the allowed target branches.

More detail: https://github.com/ecmwf-ifs/prtk

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.

2 participants