Skip to content

feat: include complytime config file for branch protection#439

Open
marcusburghardt wants to merge 1 commit intocomplytime:mainfrom
marcusburghardt:complytime-conf
Open

feat: include complytime config file for branch protection#439
marcusburghardt wants to merge 1 commit intocomplytime:mainfrom
marcusburghardt:complytime-conf

Conversation

@marcusburghardt
Copy link
Contributor

Summary

This file will be used with the reusable workflow being created in complytime/org-infra#149

Related Issues

  • Automate compliance checks for branch protection

Review Hints

@marcusburghardt marcusburghardt requested a review from a team as a code owner March 26, 2026 09:37
qduanmu
qduanmu previously approved these changes Mar 26, 2026
Copy link
Member

@qduanmu qduanmu left a comment

Choose a reason for hiding this comment

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

LGTM

complytime.yaml Outdated
variables:
url: https://github.com/complytime/complyctl
specs: builtin:github/branch-rules.yaml
access_token: ${GITHUB_TOKEN}
Copy link
Member

@AlexXuan233 AlexXuan233 Mar 26, 2026

Choose a reason for hiding this comment

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

I think we do not need this on github side, my test repo complytime.yaml . I know this came from our gitlab ci, i wondering should we keep consistent this too. But i'm ok with this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just removed since it is not necessary. Thanks @qduanmu , we will need another approval. : )

Signed-off-by: Marcus Burghardt <maburgha@redhat.com>
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