Remove pluggable-libs submodule and cleanup (#433)#447
Conversation
|
Hi. The changes here are not addressing #433. Was the intention to refer to another issue? |
Thank you for the review! The initial commits in this PR (up to commit 06edbaa) fully address issue #433 by removing the pluggable-libs submodule and all related references. The subsequent commits (7ec9916 and 3eb2663) are a follow-up to resolve CI failures we discovered when running the workflow after merging the cleanup (see PR #447). Those changes don’t introduce new behavior or patterns in the library—they simply sanitize the check_changes output so GitHub Actions can parse multi-file lists. If you’d prefer, I can split the CI fix into a separate PR targeting the workflow alone. Otherwise, this combined PR ensures both the submodule cleanup and a green build on merge. Let me know which approach you’d prefer! |
Summary
This pull request removes the obsolete
pluggable-libssubmodule from the repository and cleans up all related references and documentation, as outlined in issue #433.Changes Made
pluggable-libsfolder (already removed/skipped locally as it did not exist).pluggable-libsentry from.gitmodules.pluggable-libsfrom:README.mdResolution
This PR fully resolves issue #433 by eliminating the deprecated submodule and all associated traces.