cleanup #2258

Merged

mfreeman451 merged 1 commit from refs/pull/2258/head into main

2025-10-02 07:07:41 +00:00

mfreeman451 commented

2025-10-02 07:07:00 +00:00

(Migrated from github.com)

Owner

Imported from GitHub pull request.

Original GitHub pull request: #1680
Original author: @mfreeman451
Original URL: https://github.com/carverauto/serviceradar/pull/1680
Original created: 2025-10-02T07:07:00Z
Original updated: 2025-10-02T07:08:03Z
Original head: carverauto/serviceradar:cleanup/bazel_builds
Original base: main
Original merged: 2025-10-02T07:07:41Z by @mfreeman451

PR Type

Other

Description

Remove obsolete documentation files
Clean up Bazel build configuration files
Delete unused shell scripts and markdown docs

Diagram Walkthrough

flowchart LR
  A["Repository"] --> B["Remove docker-up.sh"]
  A --> C["Delete OCaml build docs"]
  A --> D["Remove SRQL alignment plan"]
  A --> E["Delete tools_opam alternatives"]
  A --> F["Remove Kong build plan"]

File Walkthrough

Relevant files

Miscellaneous

docker-up.sh

Delete Docker Compose launcher script

docker-up.sh

Remove entire Docker Compose launcher script
Delete ServiceRadar stack management functionality

+0/-105

Documentation

REMOTE_OCAML_BUILD_SUCCESS.md `Remove OCaml build documentation` REMOTE_OCAML_BUILD_SUCCESS.md Remove OCaml remote build success documentation Delete BuildBuddy RBE configuration details	+0/-105
SRQL_OCSF_ALIGNMENT_PLAN.md `Delete SRQL alignment planning document` SRQL_OCSF_ALIGNMENT_PLAN.md Delete SRQL to OCSF alignment planning document Remove schema migration and query language evolution plans	+0/-389
TOOLS_OPAM_VENDORING_ALTERNATIVES.md `Remove tools_opam vendoring documentation` TOOLS_OPAM_VENDORING_ALTERNATIVES.md Remove tools_opam vendoring alternatives documentation Delete patch management and implementation strategies	+0/-747
kong_plan.md `Remove Kong build planning document` kong_plan.md Delete Kong OSS RPM build planning document Remove Bazel build configuration and packaging details	+0/-73

Imported from GitHub pull request. Original GitHub pull request: #1680 Original author: @mfreeman451 Original URL: https://github.com/carverauto/serviceradar/pull/1680 Original created: 2025-10-02T07:07:00Z Original updated: 2025-10-02T07:08:03Z Original head: carverauto/serviceradar:cleanup/bazel_builds Original base: main Original merged: 2025-10-02T07:07:41Z by @mfreeman451 --- ### **PR Type** Other ___ ### **Description** - Remove obsolete documentation files - Clean up Bazel build configuration files - Delete unused shell scripts and markdown docs ___ ### Diagram Walkthrough ```mermaid flowchart LR A["Repository"] --> B["Remove docker-up.sh"] A --> C["Delete OCaml build docs"] A --> D["Remove SRQL alignment plan"] A --> E["Delete tools_opam alternatives"] A --> F["Remove Kong build plan"] ``` <details> <summary><h3> File Walkthrough</h3></summary> <table><thead><tr><th></th><th align="left">Relevant files</th></tr></thead><tbody><tr><td><strong>Miscellaneous</strong></td><td><table> <tr> <td> <details> <summary><strong>docker-up.sh</strong><dd><code>Delete Docker Compose launcher script</code>                                        </dd></summary> <hr> docker-up.sh <ul><li>Remove entire Docker Compose launcher script<br> <li> Delete ServiceRadar stack management functionality</ul> </details> </td> <td><a href="https://github.com/carverauto/serviceradar/pull/1680/files#diff-769de22ba874f720cfdc8d53fc5fb14b9a6694db56aef72850d6b55e1fb884be">+0/-105</a>  </td> </tr> </table></td></tr><tr><td><strong>Documentation</strong></td><td><table> <tr> <td> <details> <summary><strong>REMOTE_OCAML_BUILD_SUCCESS.md</strong><dd><code>Remove OCaml build documentation</code>                                                  </dd></summary> <hr> REMOTE_OCAML_BUILD_SUCCESS.md <ul><li>Remove OCaml remote build success documentation<br> <li> Delete BuildBuddy RBE configuration details</ul> </details> </td> <td><a href="https://github.com/carverauto/serviceradar/pull/1680/files#diff-4ce7d608c2d14e3ac50565a7e95a1254b57208767e039b1943ce43c74898edf5">+0/-105</a>  </td> </tr> <tr> <td> <details> <summary><strong>SRQL_OCSF_ALIGNMENT_PLAN.md</strong><dd><code>Delete SRQL alignment planning document</code>                                    </dd></summary> <hr> SRQL_OCSF_ALIGNMENT_PLAN.md <ul><li>Delete SRQL to OCSF alignment planning document<br> <li> Remove schema migration and query language evolution plans</ul> </details> </td> <td><a href="https://github.com/carverauto/serviceradar/pull/1680/files#diff-488bbb4a630bb43a568e9f984b9f4abc899bc1aeafbd1082094eb00ad95c26fd">+0/-389</a>  </td> </tr> <tr> <td> <details> <summary><strong>TOOLS_OPAM_VENDORING_ALTERNATIVES.md</strong><dd><code>Remove tools_opam vendoring documentation</code>                                </dd></summary> <hr> TOOLS_OPAM_VENDORING_ALTERNATIVES.md <ul><li>Remove tools_opam vendoring alternatives documentation<br> <li> Delete patch management and implementation strategies</ul> </details> </td> <td><a href="https://github.com/carverauto/serviceradar/pull/1680/files#diff-297134c1d03b39bde3cf067d550c78dd3d84b1d50b08c060403a5554bf522d7e">+0/-747</a>  </td> </tr> <tr> <td> <details> <summary><strong>kong_plan.md</strong><dd><code>Remove Kong build planning document</code>                                            </dd></summary> <hr> kong_plan.md <ul><li>Delete Kong OSS RPM build planning document<br> <li> Remove Bazel build configuration and packaging details</ul> </details> </td> <td><a href="https://github.com/carverauto/serviceradar/pull/1680/files#diff-b20f213badc824d85ec79bec44af9132dc78fbc1734bea0e8b5c3ba20f9c5636">+0/-73</a>    </td> </tr> </table></td></tr></tr></tbody></table> </details> ___

qodo-code-review[bot] commented

2025-10-02 07:08:02 +00:00

(Migrated from github.com)

Author

Owner

Imported GitHub PR comment.

Original author: @qodo-code-review[bot]
Original URL: https://github.com/carverauto/serviceradar/pull/1680#issuecomment-3359496468
Original created: 2025-10-02T07:08:02Z

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢	No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
⚪	🎫 No ticket provided Create ticket/issue
Codebase Duplication Compliance
⚪	Codebase context is not defined Follow the guide to enable codebase context checks.
Custom Compliance
⚪	No custom compliance provided Follow the guide to enable custom compliance check.

Compliance status legend

🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

Imported GitHub PR comment. Original author: @qodo-code-review[bot] Original URL: https://github.com/carverauto/serviceradar/pull/1680#issuecomment-3359496468 Original created: 2025-10-02T07:08:02Z --- ## PR Compliance Guide 🔍  Below is a summary of compliance checks for this PR:<br> <table><tbody><tr><td colspan='2'><strong>Security Compliance</strong></td></tr> <tr><td>🟢</td><td><details><summary><strong>No security concerns identified</strong></summary> No security vulnerabilities detected by AI analysis. Human verification advised for critical code. </details></td></tr> <tr><td colspan='2'><strong>Ticket Compliance</strong></td></tr> <tr><td>⚪</td><td><details><summary>🎫 <strong>No ticket provided </summary></strong> - [ ] Create ticket/issue  </details></td></tr> <tr><td colspan='2'><strong>Codebase Duplication Compliance</strong></td></tr> <tr><td>⚪</td><td><details><summary><strong>Codebase context is not defined </strong></summary> Follow the <a href='https://qodo-merge-docs.qodo.ai/core-abilities/rag_context_enrichment/'>guide</a> to enable codebase context checks. </details></td></tr> <tr><td colspan='2'><strong>Custom Compliance</strong></td></tr> <tr><td>⚪</td><td><details><summary><strong>No custom compliance provided</strong></summary> Follow the <a href='https://qodo-merge-docs.qodo.ai/tools/compliance/'>guide</a> to enable custom compliance check. </details></td></tr> <tr><td align="center" colspan="2">   </td></tr></tbody></table> <details><summary>Compliance status legend</summary> 🟢 - Fully Compliant<br> 🟡 - Partial Compliant<br> 🔴 - Not Compliant<br> ⚪ - Requires Further Human Verification<br> 🏷️ - Compliance label<br> </details>

qodo-code-review[bot] commented

2025-10-02 07:08:03 +00:00

(Migrated from github.com)

Author

Owner

Imported GitHub PR comment.

Original author: @qodo-code-review[bot]
Original URL: https://github.com/carverauto/serviceradar/pull/1680#issuecomment-3359496546
Original created: 2025-10-02T07:08:03Z

PR Code Suggestions ✨

No code suggestions found for the PR.

Imported GitHub PR comment. Original author: @qodo-code-review[bot] Original URL: https://github.com/carverauto/serviceradar/pull/1680#issuecomment-3359496546 Original created: 2025-10-02T07:08:03Z --- ## PR Code Suggestions ✨ No code suggestions found for the PR.