Skip to content

Revert organization license audit - #42

Open
kronk-bot[bot] wants to merge 1 commit into
mainfrom
revert/22271-org-license-audit
Open

Revert organization license audit#42
kronk-bot[bot] wants to merge 1 commit into
mainfrom
revert/22271-org-license-audit

Conversation

@kronk-bot

@kronk-bot kronk-bot Bot commented Sep 8, 2026

Copy link
Copy Markdown

[written by AI]

Summary

Replacement internal governance automation and policy are implemented in PickNikRobotics/moveit_pro#22471. This PR only restores the public customer-facing repository.

Validation

  • candidate tree hash exactly matches Audit public repository root licenses across PickNik orgs #41's first-parent tree
  • existing reusable workflows, find_release_branch, README, and LICENSE remain byte-identical
  • no organization-audit identifiers remain
  • git diff --check
  • independent review approved with no blockers

…rg-license-audit"

This reverts commit 5d77dd1, reversing
changes made to e291b03.
@kronk-bot
kronk-bot Bot requested a review from rlpratt12 September 8, 2026 22:42
@coderabbitai

coderabbitai Bot commented Sep 8, 2026

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 82925380-1aff-4ce1-ba78-fd874a02c63f

📥 Commits

Reviewing files that changed from the base of the PR and between 5d77dd1 and 7019a43.

📒 Files selected for processing (5)
  • .github/workflows/public_repo_license_audit.yaml
  • README.md
  • scripts/audit_public_repo_licenses.sh
  • scripts/public_repo_license_findings.jq
  • scripts/test_audit_public_repo_licenses.py
💤 Files with no reviewable changes (5)
  • .github/workflows/public_repo_license_audit.yaml
  • scripts/audit_public_repo_licenses.sh
  • README.md
  • scripts/public_repo_license_findings.jq
  • scripts/test_audit_public_repo_licenses.py

Included review availability: 9 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 10 reviews per hour.


📝 Summary

Summary by CodeRabbit

  • Removed

    • Removed automated public repository license auditing and its scheduled, pull-request, and manual checks.
    • Removed related license-finding, issue-management, and reporting capabilities.
    • Removed local scripts and tests supporting the audit process.
  • Documentation

    • Removed README documentation and usage instructions for the public repository license audit.

Walkthrough

Changes

Public repository license audit removal

Layer / File(s) Summary
Remove audit implementation
scripts/audit_public_repo_licenses.sh, scripts/public_repo_license_findings.jq
The repository scanning script and its jq findings filter were deleted.
Remove workflow and documentation
.github/workflows/public_repo_license_audit.yaml, README.md
The audit workflow and its README documentation were deleted.
Remove audit tests
scripts/test_audit_public_repo_licenses.py
The mock GitHub test module and its audit and issue-management tests were deleted.

Priority: ⬇️ Low

Merge Risk: ⚪ Minimal · up to 7019a

This change removes the public license-audit automation and its related workflow, tests, and documentation. No current merge-readiness risk is identified from the supplied evidence.


Caution

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

  • Ignore (reviewers only)

❌ Failed checks (1 error)

Check name Status Explanation Resolution
Human Review Check ❌ Error The PR deletes a GitHub Actions CI/CD workflow and an executable launch script under scripts/. The deleted workflow also contains permissions declarations and uses GH_TOKEN. These are explicit f… This PR requires review by a requested human reviewer. After review, a non-author requested reviewer should override this pre-merge check.
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description directly explains that the pull request reverts PR #41 and removes internal organization-governance automation. This matches the deleted workflow, scripts, tests, and README documentat…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Human Review Check

Explanation

The PR deletes a GitHub Actions CI/CD workflow and an executable launch script under scripts/. The deleted workflow also contains permissions declarations and uses GH_TOKEN. These are explicit failure categories in the check. The change is causal because the PR removes the workflow triggers, runner jobs, permissions, and script that performed the audit.

  • Fix all pre-merge checks with AI

Comment @coderabbitai help to get the list of available commands.

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.

0 participants