Skip to content

chore: include complyapi in safe-settings config - #210

Merged
marcusburghardt merged 2 commits into
complytime:mainfrom
marcusburghardt:ruleset_complyapi
Sep 4, 2026
Merged

chore: include complyapi in safe-settings config#210
marcusburghardt merged 2 commits into
complytime:mainfrom
marcusburghardt:ruleset_complyapi

Conversation

@marcusburghardt

Copy link
Copy Markdown
Member

Summary

New code related repository in ComplyTime org.

Related Issues

Review Hints

  • It includes a new repository in a existing process to manage branch protection rules.

New code related repository in complytime org.

Signed-off-by: Marcus Burghardt <maburgha@redhat.com>
jflowers
jflowers previously approved these changes Sep 3, 2026

@jflowers jflowers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

gxmiranda
gxmiranda previously approved these changes Sep 3, 2026

@gxmiranda gxmiranda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

The boundary test requires repos in the ruleset
repository_name.include to also be in the matching suborg file.
Add complyapi to suborgs/code-repos.yml so the boundary test
passes.

Also add complyapi to deployment-settings.yml restrictedRepos
include list, otherwise safe-settings skips it entirely.

Assisted-by: OpenCode (claude-opus-4-6)
Signed-off-by: Marcus Burghardt <maburgha@redhat.com>
@marcusburghardt

Copy link
Copy Markdown
Member Author

@jflowers and @gxmiranda I missed other files to update and included a second commit. Thanks

@marcusburghardt
marcusburghardt enabled auto-merge (rebase) September 3, 2026 14:57

@gxmiranda gxmiranda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@marcusburghardt
marcusburghardt merged commit 22f0527 into complytime:main Sep 4, 2026
13 checks passed
@marcusburghardt
marcusburghardt deleted the ruleset_complyapi branch September 4, 2026 08:09
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.

4 participants