Skip to content

CID-21024 Get repo ready to go public#249

Merged
btr-rmconsole-4[bot] merged 2 commits into
masterfrom
CID-21024
May 8, 2026
Merged

CID-21024 Get repo ready to go public#249
btr-rmconsole-4[bot] merged 2 commits into
masterfrom
CID-21024

Conversation

@zanehala-wk
Copy link
Copy Markdown
Contributor

@zanehala-wk zanehala-wk commented May 7, 2026

Before we can take this repo public we need to remove some of the things that rely on our self-hosted runners.

  • Removed references to artifactory
  • Removed renovate workflow, since it relies on a secret that is only allowed on private repos
  • Removed internally interacting actions

@btr-rmconsole-7
Copy link
Copy Markdown

btr-rmconsole-7 Bot commented May 7, 2026

Merge Requirements Met ✅

Request Rosie to automerge this pull request by including @Workiva/release-management-p in a comment.

General Information

Ticket(s):

Code Review(s): #249

Reviewers: andrewlundberg-wf

Additional Information

Watchlist Notifications: None


Note: This is a shortened report. Click here to view Rosie's full evaluation.
Click here for documentation on the merge-requirements status check
Last updated on Friday, May 08 11:30 AM CST

@btr-github-actions
Copy link
Copy Markdown

btr-github-actions Bot commented May 7, 2026

Security Audit Results

Please direct questions to #support-infosec.

Commit Signing

✅ All commits are signed

Raven

✅ 40 global file checks pass
✅ 94 global keyword checks pass

Images

✅ No Dockerfiles in this PR

Workflows

.github/workflows/gha-security-scanner.yaml found.

@zanehala-wk zanehala-wk marked this pull request as ready for review May 7, 2026 21:11
@zanehala-wk zanehala-wk requested review from a team as code owners May 7, 2026 21:11
Comment thread .github/workflows/ci.yml
@@ -1,28 +0,0 @@
name: Run renovate on k8s-actions-runner
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.

Will this mean we have to come up with a new method for keeping semgrep updated

@micahbeeman-wf
Copy link
Copy Markdown
Contributor

The things we're removing here will still be visible in the git history, is that a concern?

@chrisgustavsen-wf
Copy link
Copy Markdown
Contributor

I wouldn't think so, we are leaving the entirety of the git history intact. Nothing is being removed to go public.

Copy link
Copy Markdown
Contributor

@andrewlundberg-wf andrewlundberg-wf left a comment

Choose a reason for hiding this comment

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

@Workiva/release-management-pp

@btr-rmconsole-4
Copy link
Copy Markdown

@andrewlundberg-wf I will not merge this because:

  • 'Dependency Scan Clean' is False
  • dependencies not scanned

Copy link
Copy Markdown

@rmconsole-wf rmconsole-wf left a comment

Choose a reason for hiding this comment

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

+1 from RM

@btr-rmconsole-4 btr-rmconsole-4 Bot merged commit 30de60d into master May 8, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants