Skip to content

Remove obsolete command registry#5398

Merged
greenbonebot merged 2 commits into
mainfrom
remove-command-registry
Jul 3, 2026
Merged

Remove obsolete command registry#5398
greenbonebot merged 2 commits into
mainfrom
remove-command-registry

Conversation

@bjoernricks

Copy link
Copy Markdown
Contributor

What

Remove obsolete command registry

Why

With all commands converted to TypeScript and instantiated in the Gmp class directly the command registry is unused. Therefore it can be removed.

With all commands converted to TypeScript and instantiated in the Gmp
class directly the command registry is unused. Therefore it can be
removed.
@bjoernricks bjoernricks requested a review from a team as a code owner July 2, 2026 09:51
@greenbonebot greenbonebot enabled auto-merge (rebase) July 2, 2026 09:51
@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 97e06bc.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@greenbonebot greenbonebot merged commit df3d5d8 into main Jul 3, 2026
25 checks passed
@greenbonebot greenbonebot deleted the remove-command-registry branch July 3, 2026 06:49
@codecov

codecov Bot commented Jul 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.62%. Comparing base (7f98d70) to head (97e06bc).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5398   +/-   ##
=======================================
  Coverage   80.61%   80.62%           
=======================================
  Files        1269     1268    -1     
  Lines       33075    33063   -12     
  Branches     9921     9921           
=======================================
- Hits        26663    26656    -7     
+ Misses       5723     5718    -5     
  Partials      689      689           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

3 participants