Skip to content

Inline helper methods to the npm endpoint classes

What does this MR do and why?

This is a follow-up to !154577 (merged), and a preparation for tackling #463822.

Like the earlier MR, this is also a refactoring to remove endpoint_scope-related code branching in the code. For this MR, we're moving methods in the shared helper module into dedicated helper methods of the individual endpoint (project, group, instance) classes. This lets us remove if endpoint == ... branching logic in the code.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

No UI changes 🌈

How to set up and validate locally

Check for regressions in npm install (uses the metadata endpoint) for all three levels:

NOTE: You'll need to install a package at the appropriate level first

Related to #464320 (closed)

Edited by Radamanthus Batnag

Merge request reports

Loading